Re: [PATCH] status: do not get confused by submodules in excluded directories

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Oct 25, 2017, 13:26 UTC
Message-ID: <alpine.DEB.2.21.1.1710251522530.6482@virtualbox>
In-Reply-To: <20171024082029.GB7987@alpha.vpn.ikke.info>

Hi Kevin,

On Tue, 24 Oct 2017, Kevin Daudt wrote:

Show 21 quoted lines

> On Tue, Oct 17, 2017 at 03:10:11PM +0200, Johannes Schindelin wrote:
> > diff --git a/t/t7061-wtstatus-ignore.sh b/t/t7061-wtstatus-ignore.sh
> > index fc6013ba3c8..8c849a4cd2f 100755
> > --- a/t/t7061-wtstatus-ignore.sh
> > +++ b/t/t7061-wtstatus-ignore.sh
> > @@ -272,4 +272,18 @@ test_expect_success 'status ignored tracked directory with uncommitted file in t
> >  	test_cmp expected actual
> >  '
> >  
> > +cat >expected <<\EOF
> > +!! tracked/submodule/
> > +EOF
> > +
> > +test_expect_success 'status ignores submodule in excluded directory' '
> > +	git init tracked/submodule &&
> > +	(
> > +		cd tracked/submodule &&
> > +		test_commit initial
> > +	) &&
> 
> Could this use test_commit -C tracked/submodule initial?

Yes! Thanks. For some reason, I did not even think that test_commit would accept the -C option.

Ciao, Dscho