summaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorOndrej Oprala <ooprala@redhat.com>2013-02-22 13:48:57 +0100
committerBernhard Voelker <mail@bernhard-voelker.de>2013-02-22 21:31:32 +0100
commit3a20f6888575be7059e9acac07d397009e98c213 (patch)
treef550329b8763bd36cf2b4c7c4775873169795e28 /tests
parent6229654741554561952c6106f75681e4382cc890 (diff)
downloadcoreutils-3a20f6888575be7059e9acac07d397009e98c213.tar.xz
install: cleanup properly if the strip program failed for any reason
* src/install.c (strip): Indicate failure with a return code instead of terminating the program. (install_file_in_file): Handle strip's return code and unlink the created file if necessary. * tests/install/strip-program.sh: Add a test to cover the changes. * NEWS (Bug fixes): Mention the fix. Reported by John Reiser in http://bugzilla.redhat.com/632444.
Diffstat (limited to 'tests')
-rwxr-xr-xtests/install/strip-program.sh4
1 files changed, 4 insertions, 0 deletions
diff --git a/tests/install/strip-program.sh b/tests/install/strip-program.sh
index 8950d50cb..5d6537303 100755
--- a/tests/install/strip-program.sh
+++ b/tests/install/strip-program.sh
@@ -33,4 +33,8 @@ echo aBc > exp || fail=1
ginstall src dest -s --strip-program=./b || fail=1
compare exp dest || fail=1
+# Check that install cleans up properly if strip fails.
+ginstall src dest2 -s --strip-program=./FOO && fail=1
+test -e dest2 && fail=1
+
Exit $fail