build: Fix for bash quoting in supermin-split.sh.in (RHBZ#566511 RHBZ#566512)

[libguestfs.git] / appliance / supermin-split.sh.in

diff --git a/appliance/supermin-split.sh.in b/appliance/supermin-split.sh.in
index fe4ab6e..753fca3 100755 (executable)
--- a/appliance/supermin-split.sh.in
+++ b/appliance/supermin-split.sh.in
@@ -51,43 +51,62 @@ for path in $(find -not -name fakeroot.log); do
     dir=$(dirname "$path")
     file=$(basename "$path")
 
+    # For quoting problems with the bash =~ operator, see bash FAQ
+    # question E14 here http://tiswww.case.edu/php/chet/bash/FAQ and
+    # http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=487387#25
+    # (RHBZ#566511).
+
     # All we're going to keep are the special files /init, the daemon,
     # configuration files (/etc), devices and modifiable stuff (/var).
     if [ "$path" = "./init" -o "$file" = "guestfsd" ]; then
-       echo "$path" >&5
+        echo "$path" >&5
 
-    elif [[ "$path" =~ '^\./etc' || "$path" =~ '^./dev' || "$path" =~ '^\./var' ]]; then
-       echo "$path" >&5
+    elif [[ "$path" =~ ^\./etc || "$path" =~ ^\./dev || "$path" =~ ^\./var ]]
+    then
+        echo "$path" >&5
 
     # Kernel modules are always copied in from the host, including all
     # the dependency files.
-    elif [[ "$path" =~ '^\./lib/modules/' ]]; then
-       :
+    elif [[ "$path" =~ ^\./lib/modules/ ]]; then
+        :
+
+    # On mock/Koji, exclude bogus /builddir directory which for some
+    # reason contains some yum temporary files (RHBZ#566512).
+    elif [[ "$path" =~ ^\./builddir ]]; then
+        :
 
     elif [ -d "$path" ]; then
-       # Always write directory names to both output files.
-       echo "$path" >&5
-       echo "$path" >&6
+        # Always write directory names to both output files.
+        echo "$path" >&5
+        echo "$path" >&6
 
     # Some libraries need fixed version numbers replaced by wildcards.
 
-    elif [[ "$file" =~ '^ld-[.0-9]+\.so$' ]]; then
-       echo "$dir/ld-*.so" >&6
+    elif [[ "$file" =~ ^ld-[.0-9]+\.so$ ]]; then
+        echo "$dir/ld-*.so" >&6
+
+    # Special case for libbfd
+    elif [[ "$file" =~ ^libbfd-.*\.so$ ]]; then
+        echo "$dir/libbfd-*.so" >&6
+
+    # Special case for libgcc_s-<gccversion>-<date>.so.N
+    elif [[ "$file" =~ ^libgcc_s-.*\.so\.([0-9]+)$ ]]; then
+        echo "$dir/libgcc_s-*.so.${BASH_REMATCH[1]}" >&6
 
     # libfoo-1.2.3.so
-    elif [[ "$file" =~ '^lib(.*)-[-.0-9]+\.so$' ]]; then
-       echo "$dir/lib${BASH_REMATCH[1]}-*.so" >&6
+    elif [[ "$file" =~ ^lib(.*)-[-.0-9]+\.so$ ]]; then
+        echo "$dir/lib${BASH_REMATCH[1]}-*.so" >&6
 
     # libfoo-1.2.3.so.1.2.3 (but NOT '*.so.N')
-    elif [[ "$file" =~ '^lib(.*)-[-.0-9]+\.so\.([0-9]+)\.' ]]; then
-       echo "$dir/lib${BASH_REMATCH[1]}-*.so.${BASH_REMATCH[2]}.*" >&6
+    elif [[ "$file" =~ ^lib(.*)-[-.0-9]+\.so\.([0-9]+)\. ]]; then
+        echo "$dir/lib${BASH_REMATCH[1]}-*.so.${BASH_REMATCH[2]}.*" >&6
 
     # libfoo.so.1.2.3 (but NOT '*.so.N')
-    elif [[ "$file" =~ '^lib(.*)\.so\.([0-9]+)\.' ]]; then
-       echo "$dir/lib${BASH_REMATCH[1]}.so.${BASH_REMATCH[2]}.*" >&6
+    elif [[ "$file" =~ ^lib(.*)\.so\.([0-9]+)\. ]]; then
+        echo "$dir/lib${BASH_REMATCH[1]}.so.${BASH_REMATCH[2]}.*" >&6
 
     else
-       # Anything else comes from the host directly.
-       echo "$path" >&6
+        # Anything else comes from the host directly.
+        echo "$path" >&6
     fi
 done