# HG changeset patch # User Matt Mackall # Date 1333306751 18000 # Node ID 42e95631887d82762f025f4eb33143b725c91db9 # Parent a5a3af000e0d04e94df7fb99a574ef45404e6c57 tests: remove sed -i from test-record diff -r a5a3af000e0d -r 42e95631887d contrib/check-code.py --- a/contrib/check-code.py Sat Mar 31 15:39:44 2012 -0500 +++ b/contrib/check-code.py Sun Apr 01 13:59:11 2012 -0500 @@ -47,6 +47,7 @@ (r'\W\$?\(\([^\)\n]*\)\)', "don't use (()) or $(()), use 'expr'"), (r'^function', "don't use 'function', use old style"), (r'grep.*-q', "don't use 'grep -q', redirect to /dev/null"), + (r'sed.*-i', "don't use 'sed -i', use a temporary file"), (r'echo.*\\n', "don't use 'echo \\n', use printf"), (r'echo -n', "don't use 'echo -n', use printf"), (r'^diff.*-\w*N', "don't use 'diff -N'"), diff -r a5a3af000e0d -r 42e95631887d tests/test-record.t --- a/tests/test-record.t Sat Mar 31 15:39:44 2012 -0500 +++ b/tests/test-record.t Sun Apr 01 13:59:11 2012 -0500 @@ -919,7 +919,8 @@ $ cat > editor << '__EOF__' > #!/bin/sh - > sed -i.bak -e 7d -e '5s/^-/ /' "$1" + > sed -e 7d -e '5s/^-/ /' "$1" > tmp + > mv tmp "$1" > __EOF__ $ chmod +x editor $ cat > editedfile << '__EOF__' @@ -980,7 +981,8 @@ $ echo "This line has been added" >> editedfile $ cat > editor << '__EOF__' > #!/bin/sh - > sed -i -e 's/^[-+]/ /' "$1" + > sed -e 's/^[-+]/ /' "$1" > tmp + > mv tmp "$1" > __EOF__ $ chmod +x editor $ HGEDITOR="'`pwd`'"/editor hg record < tmp + $ mv tmp editedfile $ echo "This line has been added" >> editedfile $ cat > editor << '__EOF__' > #!/bin/sh - > sed -i s/This/That/ "$1" + > sed s/This/That/ "$1" > tmp + > mv tmp "$1" > __EOF__ $ chmod +x editor $ HGEDITOR="'`pwd`'"/editor hg record <