No need for HEADER.cgi to be writable by the web server

[chef.git] / cookbooks / planet / templates / default / planetdump.erb

diff --git a/cookbooks/planet/templates/default/planetdump.erb b/cookbooks/planet/templates/default/planetdump.erb
index dff206b35a2b0a9c271065711c850ffaa1aeda3c..3670c72a0e1365a0fba5a8a581a809cb894ab079 100644 (file)
--- a/cookbooks/planet/templates/default/planetdump.erb
+++ b/cookbooks/planet/templates/default/planetdump.erb
@@ -19,10 +19,10 @@ date="${year:2:2}${BASH_REMATCH[2]}${BASH_REMATCH[3]}"
 # Check the lock
 if [ -f /tmp/planetdump.lock ]; then
     if [ "$(ps -p `cat /tmp/planetdump.lock` | wc -l)" -gt 1 ]; then
 # Check the lock
 if [ -f /tmp/planetdump.lock ]; then
     if [ "$(ps -p `cat /tmp/planetdump.lock` | wc -l)" -gt 1 ]; then

-       echo "Error: Another planetdump is running"
-       exit 1
+        echo "Error: Another planetdump is running"
+        exit 1

     else
     else

-       rm /tmp/planetdump.lock
+        rm /tmp/planetdump.lock

     fi
 fi
 
     fi
 fi
 


@@ -31,18 +31,25 @@ fi
 # and incron doesn't yet support MAILTO like cron does. The
 # command below appears to work in bash as well as dash.
 logfile="/tmp/planetdump.log.$$"
 # and incron doesn't yet support MAILTO like cron does. The
 # command below appears to work in bash as well as dash.
 logfile="/tmp/planetdump.log.$$"

-exec > $logfile 2>&1
+exec > "${logfile}" 2>&1

 
 

-# Create Lock
+# Create lock file

 echo $$ > /tmp/planetdump.lock
 
 # Define cleanup function
 function cleanup {
 echo $$ > /tmp/planetdump.lock
 
 # Define cleanup function
 function cleanup {

+    # Remove the lock file

     rm /tmp/planetdump.lock
     rm /tmp/planetdump.lock

+

     # Send an email with the output, since incron doesn't yet
     # Send an email with the output, since incron doesn't yet

-    # support doing this in the incrontab.
-    /usr/bin/mailx -s "Planet dump output: ${file}" zerebubuth@gmail.com < $logfile
-    rm $logfile
+    # support doing this in the incrontab
+    if [[ -s "$logfile" ]]
+    then
+        mailx -s "Planet dump output: ${file}" zerebubuth@gmail.com < "${logfile}"
+    fi
+
+    # Remove the log file
+    rm -f "${logfile}"

 }
 
 # Remove lock on exit
 }
 
 # Remove lock on exit