Re: [PATCH v2 3/8] config: Use parseopt.
- From
Felipe Contreras <felipe.contreras@gmail.com>
- Date
- Feb 17, 2009, 13:21 UTC
- Message-ID
- <94a0d4530902170521w71127810y188d27faaf1f2b83@mail.gmail.com>
- In-Reply-To
- <alpine.DEB.1.00.0902171255220.6185@intel-tinevez-2-302>
On Tue, Feb 17, 2009 at 1:55 PM, Johannes Schindelin <Johannes.Schindelin@gmx.de> wrote:
Show 19 quoted lines
> Hi, > > On Tue, 17 Feb 2009, Felipe Contreras wrote: > >> On Tue, Feb 17, 2009 at 7:44 AM, Junio C Hamano <gitster@pobox.com> wrote: >> > I've queued the entire series on top of fc/config-editor topic and even >> > merged the result in 'pu' once, but I had to reintegrate 'pu' without the >> > series. >> > >> > Before this commit, t/t1300-repo-config.sh passes, but this one breaks >> > the test. >> >> Ah, I didn't know there was a test for that. >> >> I've fixed most the issues but unfortunately parseopt barfs when -1 is >> used as an argument. That should be fixed somehow, otherwise this >> patch will never pass the test. > > Have you seen PARSE_OPT_STOP_AT_NON_OPTION?
Thanks, that works... cooking up a new series.
-- Felipe Contreras