Yaakov (Cygwin Ports <yselkowitz@...> writes:
> Nothing has changed wrt handling of skip:, nor do I understand why you
> find this to be an issue.

I check all downloaded setup.ini files before use and having all source-only
packages show up with an "@ empty-package" line and nothing else is an
obvious error.  Yes, it doesn't throw off setup.exe, but these lines are
simply not supposed to be there and they aren't for similarly constructed
packages in Cygwin.  This would likely be an indication that you still use
genini rather than upset.  Genini needs the follwing patch in parsedir to
deal with "skip:" correctly:

     return unless -e $setup_hint;
     parse("$setup_hint", $pname);
+    next unless exists $pkg{$pname};
     my $explicit = 0;

Additionally, the code to ignore the new keys you introduced in CVS V1.15 is
needlessly verbose, the following line would do the same thing more efficiently:

        /^(arch|release|skip):/ and next;

Hope this helps.

Regards,
Achim.


------------------------------------------------------------------------------
Infragistics Professional
Build stunning WinForms apps today!
Reboot your WinForms applications with our WinForms controls. 
Build a bridge from your legacy apps to the future.
http://pubads.g.doubleclick.net/gampad/clk?id=153845071&iu=/4140/ostg.clktrk
_______________________________________________
Cygwin-ports-general mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/cygwin-ports-general

Reply via email to