Stefan Beller <sbel...@google.com> writes: > Signed-off-by: Stefan Beller <sbel...@google.com> > --- > t/lib-submodule-update.sh | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/t/lib-submodule-update.sh b/t/lib-submodule-update.sh > index c0d6325133..00128f28b5 100755 > --- a/t/lib-submodule-update.sh > +++ b/t/lib-submodule-update.sh > @@ -193,6 +193,11 @@ test_superproject_content () { > # Test that the given submodule at path "$1" contains the content according > # to the submodule commit recorded in the superproject's commit "$2" > test_submodule_content () { > + if test x"$1" = "x-C" > + then > + cd "$2" > + shift; shift;
That's old fashoned like me ;-) It seems "shift [<n>]" is already used elsewhere in our test scripts without getting complaints. 'git-difftool-helper.sh' is the only one that uses number above 1 in the scripted Porcelains, so it is possible that an esoteric platform without a shell that understands it may be built and used _without_ passing our test suite, so I'd be wary of using it in the scripted Porcelains, but this is part of the test suite, so I would think it is OK. > + fi > if test $# != 2 > then > echo "test_submodule_content needs two arguments"