Re: [PATCHv3 1/5] t7406: simplify by using diff --name-only instead of diff --raw

2018-08-07 Thread Junio C Hamano
Elijah Newren writes: >> I think 0/5 should fix the real bug you are deliberately keeping in >> this patch, from the point of view of organization. > > You mean 5/5? And yeah, it was just a temporary thing for > organizational purposes. I meant "a thing that comes before all the other steps".

Re: [PATCHv3 1/5] t7406: simplify by using diff --name-only instead of diff --raw

2018-08-07 Thread Elijah Newren
On Tue, Aug 7, 2018 at 10:29 AM Junio C Hamano wrote: > Elijah Newren writes: > > > We can get rid of some quoted tabs and make a few tests slightly easier > > to read and edit by just asking for the names of the files modified, > > since that's all these tests were interested in anyway. > >

Re: [PATCHv3 1/5] t7406: simplify by using diff --name-only instead of diff --raw

2018-08-07 Thread Junio C Hamano
Elijah Newren writes: > We can get rid of some quoted tabs and make a few tests slightly easier > to read and edit by just asking for the names of the files modified, > since that's all these tests were interested in anyway. Technically the quoted tab was making sure that we do not mistake

[PATCHv3 1/5] t7406: simplify by using diff --name-only instead of diff --raw

2018-08-07 Thread Elijah Newren
We can get rid of some quoted tabs and make a few tests slightly easier to read and edit by just asking for the names of the files modified, since that's all these tests were interested in anyway. Signed-off-by: Elijah Newren --- t/t7406-submodule-update.sh | 8 1 file changed, 4