Re: [PATCHv3 4/5] branch: introduce --list option

From: Michael J Gruber <git@drmicha.warpmail.net>
Date: Aug 29, 2011, 06:35 UTC
Message-ID: <4E5B3317.7000207@drmicha.warpmail.net>
In-Reply-To: <7v8vqc92yp.fsf@alter.siamese.dyndns.org>

Junio C Hamano venit, vidit, dixit 29.08.2011 07:55:

Show 17 quoted lines

> Michael J Gruber <git@drmicha.warpmail.net> writes:
> 
>> +test_expect_success 'git branch --list shows local branches' '
>> +	git branch --list >actual &&
>> +	test_cmp expect actual
>> +'
>> +
>> +cat >expect <<'EOF'
>> +  branch-one
>> +  branch-two
>> +EOF
>> +test_expect_success 'git branch --list pattern shows matching local branches' '
>> +	git branch --list branch* >actual &&
>> +	test_cmp expect actual
>> +'
> 
> Does this one exclude the "* master" entry at this stage in the series?

Sheesh, that rebasing mistake (flipping the order of 4 and 5) was present in v2 already. Only recently I learned about rebase-i's "exec" and have to make it a habit to use it for step-by-step series testing.

Sorry, I'll send out v3 a bit later when I see that there are no more comments (and a full step-by-step series test has passed).

Michael