[vlc-devel] [PATCH 5/5] check_sed: improve compatibility of cleanup.

Robert Mourning robedmo.git at gmail.com
Wed May 31 01:56:04 CEST 2017


Replaces curly brace expansion with explicit enumeration.
There exists at least one "sh" implementation which does not
support the curly-brace syntax in the removed line.

Also adds quotes around $tmp expansion, in case `pwd` contains spaces.

---
 extras/tools/bootstrap | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/extras/tools/bootstrap b/extras/tools/bootstrap
index 19b08a2..1d0397e 100755
--- a/extras/tools/bootstrap
+++ b/extras/tools/bootstrap
@@ -34,7 +34,7 @@ fi
 
 check_sed() {
 tmp="`pwd`/check_sed"
-trap "rm $tmp{,-e} 2>/dev/null" EXIT
+trap "rm \"$tmp\" \"$tmp-e\" 2>/dev/null" EXIT
 echo "test file for GNU sed" > $tmp
 if ! sed -i -e 's/sed//' $tmp >/dev/null 2>&1
 then
-- 
2.7.4



More information about the vlc-devel mailing list