fixed catifexec function to actually work, also now takes arguments that are passed, thanks to a report from John Hallam

2 files changed, 8 insertions, 4 deletions

diff --git a/ChangeLog b/ChangeLog
index b56cab9..df24adc 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -47,6 +47,8 @@ version 0.9.6 -- unreleased
 	   dup handler, with the new vsnames configuration option ; (Closes: Trac#45)
 	 . add support for capturing the package debconf selection states using
 	   debconf-get-selections
+	 . fixed catifexec function to actually work, also now passes the arguments
+	   given to catifexec() to the called command (Thanks John Hallam!)
 
 version 0.9.5 -- December 2, 2007
     backupninja changes
diff --git a/handlers/sys.in b/handlers/sys.in
index 2c48883..9e6343f 100755
--- a/handlers/sys.in
+++ b/handlers/sys.in
@@ -188,10 +188,12 @@ catiffile () {
 } 
 
 catifexec () {
-   echo $HASHES >> $sysreportfile
-   echo "# $STATUS" >> $sysreportfile
-   echo $HASHES >> $sysreportfile
-   $1  >> $sysreportfile 2>&1 || info "executing of $1 failed"
+   if [ -x $1 ]; then
+      echo $HASHES >> $sysreportfile
+      echo "# $STATUS" >> $sysreportfile
+      echo $HASHES >> $sysreportfile
+      $*  >> $sysreportfile 2>&1 || info "executing of $1 failed"
+   fi
 }