Re: [PATCH 1/3] t7900-subtree.sh: fix quoting and broken && chains

2016-07-25 Thread Junio C Hamano
Eric Sunshine writes: >> @@ -73,10 +73,10 @@ join_commits() >> test_create_commit() ( >> repo=$1 >> commit=$2 > > Perhaps &&-chain the above two lines also to future-proof against > someone inserting important code somewhere above the following 'cd'. Yup, also we can have them o

Re: [PATCH 1/3] t7900-subtree.sh: fix quoting and broken && chains

2016-07-25 Thread Eric Sunshine
On Tue, Jul 26, 2016 at 12:14 AM, David Aguilar wrote: > Allow whitespace in arguments to subtree_test_create_repo. > Add missing && chains. > > Signed-off-by: David Aguilar > --- > diff --git a/contrib/subtree/t/t7900-subtree.sh > b/contrib/subtree/t/t7900-subtree.sh > @@ -16,16 +16,16 @@ expor

[PATCH 1/3] t7900-subtree.sh: fix quoting and broken && chains

2016-07-25 Thread David Aguilar
Allow whitespace in arguments to subtree_test_create_repo. Add missing && chains. Signed-off-by: David Aguilar --- contrib/subtree/t/t7900-subtree.sh | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/contrib/subtree/t/t7900-subtree.sh b/contrib/subtree/t/t7900-sub