Re: [PATCH v3 2/5] commit test: Change $PWD to $(pwd)

2014-05-27 Thread David Kastrup
Johannes Sixt writes: > That said, it is not wrong to use $(pwd) with test_set_editor, it's just > unnecessarily slow. Any shell that knows $(...) is pretty sure to have pwd as a built-in. I don't think Git will run on those kind of ancient shells reverting to /bin/pwd here. The autoconf manual

Re: [PATCH v3 2/5] commit test: Change $PWD to $(pwd)

2014-05-26 Thread Johannes Sixt
Please do not cull the Cc list. Am 5/27/2014 8:14, schrieb Jeremiah Mahler: > On Tue, May 27, 2014 at 07:46:59AM +0200, Johannes Sixt wrote: >> Am 5/26/2014 20:56, schrieb Caleb Thompson: >>> Signed-off-by: Caleb Thompson >>> --- >>> t/t7507-commit-verbose.sh | 2 +- >>> 1 file changed, 1 insert

Re: [PATCH v3 2/5] commit test: Change $PWD to $(pwd)

2014-05-26 Thread Jeremiah Mahler
On Tue, May 27, 2014 at 07:46:59AM +0200, Johannes Sixt wrote: > Am 5/26/2014 20:56, schrieb Caleb Thompson: > > Signed-off-by: Caleb Thompson > > --- > > t/t7507-commit-verbose.sh | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/t/t7507-commit-verbose.sh b/t/t7507-

Re: [PATCH v3 2/5] commit test: Change $PWD to $(pwd)

2014-05-26 Thread Eric Sunshine
On Tue, May 27, 2014 at 1:46 AM, Johannes Sixt wrote: > Am 5/26/2014 20:56, schrieb Caleb Thompson: >> Signed-off-by: Caleb Thompson >> --- >> t/t7507-commit-verbose.sh | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/t/t7507-commit-verbose.sh b/t/t7507-commit-verbose.

Re: [PATCH v3 2/5] commit test: Change $PWD to $(pwd)

2014-05-26 Thread Johannes Sixt
Am 5/26/2014 20:56, schrieb Caleb Thompson: > Signed-off-by: Caleb Thompson > --- > t/t7507-commit-verbose.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/t/t7507-commit-verbose.sh b/t/t7507-commit-verbose.sh > index 6d778ed..3b06d73 100755 > --- a/t/t7507-commit-verb