Re: [PATCH] don't allow invalid --gc options

2007-09-03 Thread Andy Dougherty
On Mon, 3 Sep 2007, Gabor Szabo wrote: > On 9/3/07, Andy Dougherty <[EMAIL PROTECTED]> wrote: > > On Sat, 1 Sep 2007, Gabor Szabo wrote: > > > > > It has biten me so let's make sure invalid --gc options are not allowed. > > > > But what if I want to work on --gc=libc? How can I override this? >

Re: [PATCH] don't allow invalid --gc options

2007-09-02 Thread Gabor Szabo
On 9/3/07, Andy Dougherty <[EMAIL PROTECTED]> wrote: > On Sat, 1 Sep 2007, Gabor Szabo wrote: > > > It has biten me so let's make sure invalid --gc options are not allowed. > > But what if I want to work on --gc=libc? How can I override this? I am not sure I understand you. If you want to work on

Re: [PATCH] don't allow invalid --gc options

2007-09-02 Thread Andy Dougherty
On Sat, 1 Sep 2007, Gabor Szabo wrote: > It has biten me so let's make sure invalid --gc options are not allowed. But what if I want to work on --gc=libc? How can I override this? -- Andy Dougherty [EMAIL PROTECTED]

[PATCH] don't allow invalid --gc options (2nd try)

2007-09-01 Thread Gabor Szabo
Index: config/auto/gc.pm === --- config/auto/gc.pm (revision 20963) +++ config/auto/gc.pm (working copy) @@ -107,7 +107,7 @@ gc_flag => '-DGC_IS_MALLOC', ); } -else { +elsif ( $gc eq 'gc' ) {

Re: [PATCH] don't allow invalid --gc options

2007-09-01 Thread Gabor Szabo
On 9/1/07, James E Keenan <[EMAIL PROTECTED]> wrote: > (Gabor: You didn't cc: the list on this, so I'm replying directly to > you. But I think the issues you are raising merit discussion on the > list. So please feel free to copy what I write below to the list.) Sure, I meant to send it to the

Re: [PATCH] don't allow invalid --gc options

2007-09-01 Thread James E Keenan
Gabor Szabo wrote: It has biten me so let's make sure invalid --gc options are not allowed. Index: lib/Parrot/Configure/Options.pm === --- lib/Parrot/Configure/Options.pm (revision 20963) +++ lib/Parrot/Configure/Options.pm

[PATCH] don't allow invalid --gc options

2007-09-01 Thread Gabor Szabo
It has biten me so let's make sure invalid --gc options are not allowed. Index: lib/Parrot/Configure/Options.pm === --- lib/Parrot/Configure/Options.pm (revision 20963) +++ lib/Parrot/Configure/Options.pm (working copy) @@ -4