Modified fines.pl to not throw warning if --out not specified
authorJ. David Bavousett <dbavousett@ptfs.com>
Thu, 25 Jun 2009 14:43:39 +0000 (10:43 -0400)
committerGalen Charlton <galen.charlton@liblime.com>
Mon, 29 Jun 2009 12:59:59 +0000 (07:59 -0500)
It appears, in Ryan's patch, that he wants to throw a warning to the log if
the directory specified in --out is not present.  (Further messages will
be given when the open-or-die occurs a few lines later.)  However, it was
throwing the warning if --out was not specified at all, which is
undesirable.  This patch modifies that bit to check for the presence of
whatever directory is going to be used, either --out, ENV{TMPDIR}, or /tmp.
As before, if the write to the directory fails for any reason--including
its' non-existence--that is handled later, but this message will help
inform the troubleshooter.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
misc/cronjobs/fines.pl

index 42498f3..65bfb7a 100755 (executable)
@@ -105,7 +105,9 @@ if($output_dir){
     $fldir = $output_dir if( -d $output_dir );
 } else {
     $fldir = $ENV{TMPDIR} || "/tmp";
-    warn "Could not write to $output_dir ... does not exist!";
+}
+if (!-d $fldir) {
+    warn "Could not write to $fldir ... does not exist!";
 }
 $filename = $dbname;
 $filename =~ s/\W//;