]> ruderich.org/simon Gitweb - config/dotfiles.git/commitdiff
crontab.d/update.sh: Minor cleanup.
authorSimon Ruderich <simon@ruderich.org>
Mon, 18 Nov 2013 12:13:18 +0000 (13:13 +0100)
committerSimon Ruderich <simon@ruderich.org>
Tue, 19 Nov 2013 01:18:18 +0000 (02:18 +0100)
crontab.d/update.sh

index 72422fc90390e3c767eddfacd5fc676045eae484..fd4a9aef77084436b7c57f0fddc3e17067baaf1e 100755 (executable)
@@ -29,10 +29,10 @@ HEADER_WARNING="# WARNING! DO NOT EDIT THIS FILE! #"
 # Don't overwrite existing crontab entries. Not race condition free but that
 # can't be helped.
 if crontab -l >/dev/null 2>&1; then
-    if crontab -l | head -n3 | grep "^$HEADER_WARNING$" >/dev/null; then
+    if crontab -l | head -n3 | grep -F -x "$HEADER_WARNING" >/dev/null; then
         :
     else
-        echo "Existing crontab entry found, please remove it manually."
+        echo 'Existing crontab entry found, please remove it manually.'
         exit 2
     fi
 fi
@@ -46,10 +46,10 @@ fi
 # `set -e` aborts when `mktemp` fails.
 CRONTAB=`mktemp --tmpdir="$DIRECTORY" update-crontab.XXXXXXXXXXXX`
 
-echo "###################################"  > "$CRONTAB"
-echo "$HEADER_WARNING"                     >> "$CRONTAB"
-echo "###################################" >> "$CRONTAB"
-echo >> "$CRONTAB"
+echo '###################################'  >"$CRONTAB"
+echo "$HEADER_WARNING"                     >>"$CRONTAB"
+echo '###################################' >>"$CRONTAB"
+echo >>"$CRONTAB"
 printf "# It was generated from '%s/*' on %s." \
     "$DIRECTORY" "`LANG=C date '+%a, %d %b %Y %H:%M:%S %z'`" >>"$CRONTAB"
 
@@ -72,13 +72,13 @@ for file in "$DIRECTORY"/crontab.*; do
     # (cron doesn't expand variables).
     START_REGEX='^# This [a-zA-Z]* is free software: you can redistribute it'
     END_REGEX='^# along with this [a-zA-Z]*.  If not, see <[^>]*>\.'
-    < "$file" sed -e "/$START_REGEX/,/$END_REGEX/ d" \
-                  -e "/^#[ ]*$/ d" \
-                  -e "/^# Copyright (C) [0-9][0-9]* / d" \
+    <"$file" sed -e "/$START_REGEX/,/$END_REGEX/ d" \
+                 -e "/^#[ ]*$/ d" \
+                 -e "/^# Copyright (C) [0-9][0-9]* / d" \
         | cat --squeeze-blank \
         | sed "/^PATH/ s:\$HOME:$HOME:g" \
         | sed 's/RAND_SLEEP \([0-9]*\)/sleep `perl -e "srand; print int rand \1;"`/' \
-        >> "$CRONTAB"
+        >>"$CRONTAB"
 done
 
 # Update crontab with the crontab.* files.