This is the mail archive of the cygwin-apps mailing list for the Cygwin project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Please try new setup exe's

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...
>>> ... 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:
>OK to commit this patch (I think I have write access to the genini 
>module in the cygwin-apps repo)?

>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.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]