This is the mail archive of the
cygwin-apps
mailing list for the Cygwin project.
Re: genini patch
On Fri, Feb 23, 2007 at 09:59:25AM +0100, Servaas Goossens wrote:
>Thanks for submitting my previous patches. I have further modified
>genini so that it generates output that is parsed without errors by
>setup.exe. I also fixed the sorting problem for file versions (ref:
>http://sourceware.org/ml/cygwin-apps/2006-09/msg00020.html).
>
>A few issues remain, but setup.exe seems to accept them:
>* genini generates one line for the non-existing db package ("@ db")
>* genini doesn't include some -src files that do exist in the official
> setup.ini. This seems to be related to external-source values in
> setup.hint. Maybe the error is in the .hint file? (e.g. gcc)
>* for postgresql genini includes a [prev] version with no files (install
> or source). These files don't exist: the error is in the setup.hint.
>
>Here's the changelog:
>
>* genini (parsedir): Use correct sorting order based on file versions
It's a lot more complicated than this if you want to get the sorting
right. Maybe the setup.cc code could be of some use.
>(get): Fix quotes in sdesc and ldesc values
It took me a couple of tries but I think I've implemented this in a
slightly cleaner manner than your patch.
>(parse): # in the middle of a line does not start a comment
Actually, yes it does. It just doesn't start a comment in a quoted
string. I haven't committed this since more work is needed.
>Don't output keys with empty values
Committed
>Fix minor typo in warning
Committed
>New categories: KDE, Perl, Python
Committed
cgf