Jon Turney
Mon Mar 14 19:06:08 GMT 2022

On 10/01/2022 18:10, Achim Gratz wrote:
> I've rebased the remaining patches on my to-upstream branch onto the
> current release of cygport:
> Note that some of these are required to correctly build and distribute
> Perl and its modules for Cygwin (which is one reason I carry these for
> several years now).

A few comments after looking at:

lib/pkg_info.cygport: implement automatic determination of the 
appropriate perl5_0xy requirement
1. In __list_deps(), this should look at the files list in $@, not at 
files in $D, as that causes it to identify a perl5_0xy dependency for 
all subpackages, irrespective of which package (if any) contains the 
files in the vendor_perl directory.

2. This only identifies the perl5_0xy requirement for packages which own 
files in the vendor_perl directory, not for packages which contain 
executables or shared libraries dynamically linked with cygperl5_xy.dll. 
  That should at least be mentioned in the patch commentary.

More information about the Cygwin-apps mailing list