Re: Please try new setup exe's

[Christopher Faylor <cgf-use-the-mailinglist-please at cygwin dot com>]

On Tue, Jul 16, 2013 at 06:23:23PM -0400, Charles Wilson wrote:
>On 7/15/2013 2:58 PM, Christopher Faylor wrote:
>> On Mon, Jul 15, 2013 at 02:32:24PM -0400, Charles Wilson wrote:
>>> What changes did you have to make to upset, to teach it about the new
>>> format? I'd like to replicate those changes in genini...
>>>
>>> 	http://cygwin.com/cgi-bin/cvsweb.cgi/~checkout~/genini/genini?rev=1.13&cvsroot=cygwin-apps
>>>
>>> ... so I can test the new setup-exe's with my to-be-uploaded packages.
>>
>> The new setup.exe will still understand old setup.ini's, just not the
>> reverse.
>>
>> However, all that I did to upset (for this particular change there were
>> a few more other changes required) was add --release and --arch options
>> to produce the setup.ini with those tags.
>>
>> Feel free to add those to genini if you want.
>
>Works fine installing from the mirrors [1] or from local repo, with the 
>following patch to genini [2]:
>
>2013-07-16  Charles Wilson  <...>
>
>     * genini: Accept 'Debug' category. Add support for
>     --release and --arch commandline options, and emit
>     the appropriate entries (default release='cygwin',
>     default arch='x86').
>
>[1] With the already-reported issue regarding empty/src-only packages.
>
>[2] This patch does not address the recently-reported issue with genini 
>that (might be?) related to the above empty/src-only thing:
>http://cygwin.com/ml/cygwin-apps/2013-07/msg00207.html
>
>OK to commit this patch (I think I have write access to the genini 
>module in the cygwin-apps repo)?
>
>-- 
>Chuck
>

>Index: genini
>===================================================================
>RCS file: /cvs/cygwin-apps/genini/genini,v
>retrieving revision 1.13
>diff -u -p -r1.13 genini
>--- genini	21 Jul 2010 15:02:30 -0000	1.13
>+++ genini	16 Jul 2013 21:55:17 -0000
>@@ -14,16 +14,26 @@ use strict;
> sub mywarn(@);
> sub myerror(@);
> sub usage();
>+sub arch_handler(@);
> 
> my @okmissing = qw'message ldesc';
> my ($outfile, $help, $recursive);
>+my $arch = 'x86';
>+my $release = 'cygwin';

If you want to keep parity with upset, then only arch is filled out
automatically.  If there is no --release then that field doesn't show
up in the generated ini file.

With that change, feel free to checkin.

cgf