Merge branch 'ak/submodule-foreach-quoting'
commitb2a0afd96a40d6cdbba96ac9735a7b489d8d563e
authorJunio C Hamano <gitster@pobox.com>
Thu, 5 Dec 2013 20:53:17 +0000 (5 12:53 -0800)
committerJunio C Hamano <gitster@pobox.com>
Thu, 5 Dec 2013 20:53:17 +0000 (5 12:53 -0800)
tree14e5dc101eef39452a09e743ed2219f9443c9400
parentdaad3aa255ec5c08f95867feaaf8f4db03346e70
parent1c4fb136dbad762c9c4350ee79c3474ae8037587
Merge branch 'ak/submodule-foreach-quoting'

A behavior change, but a worthwhile one: "git submodule foreach"
was treating its arguments as part of a single command to be
concatenated and passed to a shell, making writing buggy
scripts too easy.

This patch preserves the old "just pass it to the shell" behavior
when a single argument is passed to 'git submodule foreach' and
moves to a new "skip the shell and use the arguments passed
unmolested" behavior when more than one argument is passed.

The old behavior (always concatenating and passing to the shell)
was similar to the 'ssh' command, while the new behavior (switching
on the number of arguments) is what 'xterm -e' does.

May need more thought to make sure this change is advertised well
so that scripts that used multiple arguments but added their own
extra layer of quoting are not broken.

* ak/submodule-foreach-quoting:
  submodule foreach: skip eval for more than one argument
git-submodule.sh
t/t7407-submodule-foreach.sh