Jonathan Tan <[email protected]> writes:
> + # Tweak the push output to make the push option outside the cert
> + # different, then replay it on a fresh dst, checking that ff is not
> + # deleted.
> + sed -i "s/\\([^ ]\\)bar/\\1baz/" push &&
This is not portable. Didn't you get an error from the lint
checker?
The attached may not be enough if "push" is a binary file, though,
in which case perl may be your friend ;-)
t/t5534-push-signed.sh | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/t/t5534-push-signed.sh b/t/t5534-push-signed.sh
index 0ef6f66b5d..effe769688 100755
--- a/t/t5534-push-signed.sh
+++ b/t/t5534-push-signed.sh
@@ -152,11 +152,11 @@ test_expect_success GPG 'inconsistent push options in
signed push not allowed' '
# Tweak the push output to make the push option outside the cert
# different, then replay it on a fresh dst, checking that ff is not
# deleted.
- sed -i "s/\\([^ ]\\)bar/\\1baz/" push &&
+ sed -e "s/\\([^ ]\\)bar/\\1baz/" push >push.tweak &&
prepare_dst &&
git -C dst config receive.certnonceseed sekrit &&
git -C dst config receive.advertisepushoptions 1 &&
- git receive-pack dst <push >out &&
+ git receive-pack dst <push.tweak >out &&
git -C dst rev-parse ff &&
grep "inconsistent push options" out
'
