Reroute the output of stdout to stderr as it is just informative messages, not to be consumed by machines.
This should not regress any scripts that try to parse the current output, as the output is already internationalized and therefore unstable. Signed-off-by: Stefan Beller <[email protected]> --- git-submodule.sh | 2 +- t/t7403-submodule-sync.sh | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/git-submodule.sh b/git-submodule.sh index c86c2e5..f075924 100755 --- a/git-submodule.sh +++ b/git-submodule.sh @@ -1095,7 +1095,7 @@ cmd_sync() if git config "submodule.$name.url" >/dev/null 2>/dev/null then displaypath=$(relative_path "$prefix$sm_path") - say "$(eval_gettext "Synchronizing submodule url for '\$displaypath'")" + say >&2 "$(eval_gettext "Synchronizing submodule url for '\$displaypath'")" git config submodule."$name".url "$super_config_url" if test -e "$sm_path"/.git diff --git a/t/t7403-submodule-sync.sh b/t/t7403-submodule-sync.sh index 79bc135..93c1dfa 100755 --- a/t/t7403-submodule-sync.sh +++ b/t/t7403-submodule-sync.sh @@ -155,7 +155,7 @@ test_expect_success '"git submodule sync" should update submodule URLs - subdire git pull --no-recurse-submodules && mkdir -p sub && cd sub && - git submodule sync >../../output + git submodule sync 2>../../output ) && grep "\\.\\./submodule" output && test -d "$( @@ -186,7 +186,7 @@ test_expect_success '"git submodule sync --recursive" should update all submodul ) && mkdir -p sub && cd sub && - git submodule sync --recursive >../../output + git submodule sync --recursive 2>../../output ) && grep "\\.\\./submodule/sub-submodule" output && test -d "$( -- 2.8.0.32.g71f8beb.dirty -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
