vserver: Cleanup vserver-hashify-all script.
authorMatthijs Kooijman <matthijs@stdin.nl>
Thu, 16 Apr 2009 13:18:47 +0000 (15:18 +0200)
committerMatthijs Kooijman <matthijs@stdin.nl>
Thu, 16 Apr 2009 13:23:11 +0000 (15:23 +0200)
This adds proper quoting and uses absolute paths.

usr/local/bin/vserver-hashify-all

index 7c038c1c82d9d77564f8dc27d7c0f8ca504c0666..d1a219089cddf597e601c69ca37465d76b41dc97 100755 (executable)
@@ -1,25 +1,25 @@
 #!/bin/sh
 
 # The dir containing vserver configuration
-VSERVERS_CONF=/etc/vservers
+VSERVERS_CONF="/etc/vservers"
 
-cd $VSERVERS_CONF
-for NAME in *; do
+for VDIR in $VSERVERS_CONF/*; do
+       NAME=`basename $VDIR`
        # Check if its config dir exists
-       if [ ! -d $NAME ]; then
+       if [ ! -d "$VDIR" ]; then
                continue;
        fi
        
        echo "Hashifying vserver $NAME..."
-       sudo vserver $NAME hashify
+       sudo vserver "$NAME" hashify
 done
 
 # Change to /tmp, to minimize the chance for an error to damage the root fs
 cd /tmp
 echo "Cleaning up unused hashed files..."
-for HASH_DIR in $VSERVERS_CONF/.defaults/apps/vunify/hash/*; do
-       if [ -L $HASH_DIR ]; then
+for HASH_DIR in "$VSERVERS_CONF"/.defaults/apps/vunify/hash/*; do
+       if [ -L "$HASH_DIR" ]; then
                # All symbolic links 
-               sudo find -H $HASH_DIR -type f -links 1 -print0 | sudo xargs -0 rm
+               sudo find -H "$HASH_DIR" -type f -links 1 -print0 | sudo xargs -0 rm
        fi
 done