diff options
author | Jeff King <peff@peff.net> | 2015-03-20 06:12:55 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2015-03-20 10:20:17 -0700 |
commit | aef591a0f91a80d2b39c5344653861967d70d720 (patch) | |
tree | 470f0a16d12391b58b352f2f338f378aed06835e /t/t3417-rebase-whitespace-fix.sh | |
parent | t3600: fix &&-chain breakage for setup commands (diff) | |
download | tgif-aef591a0f91a80d2b39c5344653861967d70d720.tar.xz |
t7201: fix &&-chain breakage
One of these breakages is in setup, but one is more severe
and may miss a real test failure. These are pulled out from
the rest, though, because we also clean up a few other
anachronisms. The most interesting is the use of this
here-doc construct:
(cat >... <<EOF
...
EOF
) &&
It looks like an attempt to make the &&-chaining more
natural by letting it come at the end of the here-doc. But
the extra sub-shell is so non-idiomatic (plus the lack of
"<<-") that it ends up confusing.
Since these are just using a single line, we can accomplish
the same thing with a single printf (which also makes the
use of tab more obvious than the verbatim whitespace).
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t3417-rebase-whitespace-fix.sh')
0 files changed, 0 insertions, 0 deletions