From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Tue, 17 Feb 2009 11:55:38 GMT
Subject: Re: [PATCH v2 3/8] config: Use parseopt.
Message-ID: <alpine.DEB.1.00.0902171255220.6185@intel-tinevez-2-302>
In-Reply-To: <94a0d4530902170235g4e481e07t39a6157894dff9aa@mail.gmail.com>

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?

Ciao,
Dscho