extfs shell script fixes (audio.in)

Leonard den Ottolander leonard at den.ottolander.nl
Thu Oct 14 13:05:01 UTC 2004


Hi,

Some remarks on the SUSE extfs shell fixes regarding audio.in:

@@ -15,7 +15,7 @@
-    cdparanoia -Q -d $1 2>&1 | grep '^[ 0-9][ 0-9][ 0-9]\.' | while
true
+    cdparanoia -Q -d "$1" 2>&1 | grep '^[ 0-9][ 0-9][ 0-9]\.' | while
true

Quoting $1 is unnecessary here as no shell is spawned. This hunk can
thus be dropped.

Same is true for
-        cdparanoia -q -d $1 $TRACK $3 >/dev/null
+        cdparanoia -q -d "$1" $TRACK "$3" >/dev/null
in @@ -28,15 +28,15 @@

@@ -48,7 +48,7 @@
-  list) audiofs_list $BASE; exit 0;;
-  copyout) audiofs_copyout $BASE $3 $4; exit 0;;
+  list) audiofs_list "$BASE"; exit 0;;
+  copyout) audiofs_copyout "$BASE" "$3" "$4"; exit 0;;

$BASE, $3 and $4 don't need to be quoted either.

This makes me wonder if the quoting of $f and the use of SEDCMD in
extfs/rpm mcrpmfs_copyout() and mcrpmfs_run aren't redundant either. Any
thoughts?

Leonard.

-- 
mount -t life -o ro /dev/dna /genetic/research





More information about the mc-devel mailing list