From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from vsmx012.vodafonemail.xion.oxcs.net (vsmx012.vodafonemail.xion.oxcs.net [153.92.174.90]) by sourceware.org (Postfix) with ESMTPS id 4C3E8385DC1A for ; Tue, 7 Apr 2020 16:52:18 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 4C3E8385DC1A Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=nexgo.de Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=Stromeko@nexgo.de Received: from vsmx004.vodafonemail.xion.oxcs.net (unknown [192.168.75.198]) by mta-8-out.mta.xion.oxcs.net (Postfix) with ESMTP id 2807FF35011 for ; Tue, 7 Apr 2020 16:52:17 +0000 (UTC) Received: from Gertrud (unknown [84.160.196.178]) by mta-8-out.mta.xion.oxcs.net (Postfix) with ESMTPA id DEFE719AE57 for ; Tue, 7 Apr 2020 16:52:14 +0000 (UTC) From: Achim Gratz To: cygwin-apps@cygwin.com Subject: Re: cygport patches for consideration References: <87k12tbw5u.fsf@Rainer.invalid> Date: Tue, 07 Apr 2020 18:52:05 +0200 In-Reply-To: (Yaakov Selkowitz's message of "Mon, 06 Apr 2020 20:52:02 -0400") Message-ID: <878sj7ck7e.fsf@Rainer.invalid> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-VADE-STATUS: LEGIT X-Spam-Status: No, score=-3.4 required=5.0 tests=BAYES_00, KAM_DMARC_STATUS, RCVD_IN_DNSWL_LOW, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: cygwin-apps@cygwin.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Cygwin package maintainer discussion list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 07 Apr 2020 16:52:20 -0000 Yaakov Selkowitz writes: >> support subdirectories in CPAN download URL > > There is no need for two variables to do the same thing. I like > Reini's idea but let's call it CPAN_SUBDIR instead. What about the > attached patch 0001? Well, Reini's chimed in after he'd seen my patch and I just added that functionality on top of what was already implemented and in use by myself. I guess I can change my cygport generator instead to use CPAN_DIR when needed, but haven't got around doing so. >> pkg_info.cygport: correct search order for Perl dependencies > > Let's improve the auto-detection instead. What about the attached > patch 0002? Autodetection just doesn't work, with both false positives and negatives in spades. Your proposed change probably produces more false negatives with not much change in the false positive rate. So why bother? There are actually multiple modules on CPAN attempting that sort of thing, one slower than the other and still none of them gets it right in all cases. So your chances of doing better with a shell script are pretty much zero. All Perl distributions come with metadata that tell you the dependencies among them already, so I just need a way to transfer that exactly into the cygport file (which again gets generated from that exact metadata, not written by hand). By way of example, Cygwin has several packages that support Perl OO layers, of which there are many (Moose, Mouse, Moo, Mo=E2=80=A6) so a dependency check pulling in = all of them would end up creating a dependency chain that's at least 200 more distributions. Note that I very deliberately do not package Moose, which is huge. >> Automatically create a test release if the release string starts wit= h a literal "0" > > Nak. There is not necessarily any correlation between a -0.* release > and whether it should be test or not. Then there ought to be. :-) >> Show pkg_tag in chatter > > Pushed a slightly different approach to master. I obviously like my patch better, because it keeps that line in the output no matter what the tag is (plus colors it so it's easier to catch a left over wrong tag when scrolling through the backlog). > Not sure what the > second part of that patch is supposed to be, however, as there is no > $pkg_testrelease variable in cygport. That's a vestigial of some other patch that was ordered before this one before I created the branch, I think. >> lib/src_install.cygpart: correct test in make_etc_defaults, possibly= show diff > > The first part is ok once the commit is reworded. But when would a > user see the output of a postinstall script? When looking in setup.log.full=E2=80=A6 this output used to go to the cons= ole, but got axed quite some time ago. > What would make more sense is to have a utility akin to "rpmconf -a" > on RPM-based systems which allows the user to compare existing files > with their /etc/defaults and choose if and how to merge the > differences. Sure, but that's not cygport's business, no? Regards, Achim. --=20 +<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+ Waldorf MIDI Implementation & additional documentation: http://Synth.Stromeko.net/Downloads.html#WaldorfDocs