Re: Mini bug report origin/pu: t1512 failed on Mac OS X (commit 957d74062c1f0e ?)

2012-07-13 Thread Junio C Hamano
Stefano Lattarini writes: > On 07/12/2012 01:30 AM, Junio C Hamano wrote: >> Junio C Hamano writes: >> >>> I think the other tests in t/ prefer to unquote it so that we would >>> ignore spaces around "wc -l" output, i.e. >>> >>> test $(wc -l >> >>> Thanks for a report. >> >> -- >8 -- >> Su

Re: Mini bug report origin/pu: t1512 failed on Mac OS X (commit 957d74062c1f0e ?)

2012-07-13 Thread Stefano Lattarini
On 07/12/2012 01:30 AM, Junio C Hamano wrote: > Junio C Hamano writes: > >> I think the other tests in t/ prefer to unquote it so that we would >> ignore spaces around "wc -l" output, i.e. >> >> test $(wc -l > >> Thanks for a report. > > -- >8 -- > Subject: [PATCH] t1512: ignore whitespaces

Re: Mini bug report origin/pu: t1512 failed on Mac OS X (commit 957d74062c1f0e ?)

2012-07-12 Thread Junio C Hamano
Johannes Sixt writes: > Am 7/12/2012 1:30, schrieb Junio C Hamano: >> -test "$(wc -l > +test $(wc -l > We have a helper function for this: > > test_line_count = 16 actual && OK. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...

Re: Mini bug report origin/pu: t1512 failed on Mac OS X (commit 957d74062c1f0e ?)

2012-07-12 Thread Johannes Sixt
Am 7/12/2012 1:30, schrieb Junio C Hamano: > - test "$(wc -l + test $(wc -l http://vger.kernel.org/majordomo-info.html

Re: Mini bug report origin/pu: t1512 failed on Mac OS X (commit 957d74062c1f0e ?)

2012-07-11 Thread Junio C Hamano
Junio C Hamano writes: > I think the other tests in t/ prefer to unquote it so that we would > ignore spaces around "wc -l" output, i.e. > > test $(wc -l > Thanks for a report. -- >8 -- Subject: [PATCH] t1512: ignore whitespaces in wc -l output Some implementations of sed (e.g. MacOS X)

Re: Mini bug report origin/pu: t1512 failed on Mac OS X (commit 957d74062c1f0e ?)

2012-07-11 Thread Junio C Hamano
Torsten Bögershausen writes: > The following tweak will make t1512 work on my Mac OS box: > > > --- a/t/t1512-rev-parse-disambiguation.sh > +++ b/t/t1512-rev-parse-disambiguation.sh > @@ -257,7 +257,7 @@ test_expect_success 'rev-parse --disambiguate' ' > # commits created by commit-tree i

Mini bug report origin/pu: t1512 failed on Mac OS X (commit 957d74062c1f0e ?)

2012-07-11 Thread Torsten Bögershausen
The following tweak will make t1512 work on my Mac OS box: --- a/t/t1512-rev-parse-disambiguation.sh +++ b/t/t1512-rev-parse-disambiguation.sh @@ -257,7 +257,7 @@ test_expect_success 'rev-parse --disambiguate' ' # commits created by commit-tree in earlier tests do not share # the