Please try new setup exe's
Christopher Faylor
cgf-use-the-mailinglist-please@cygwin.com
Wed Jul 17 14:21:00 GMT 2013
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
More information about the Cygwin-apps
mailing list