Title: [opsview] [9626] Corrected umask failure capturing (starman runs with umask 0, so a simple || was causing a failure)
Revision
9626
Author
tvoon
Date
2012-07-26 17:00:27 +0100 (Thu, 26 Jul 2012)

Log Message

Corrected umask failure capturing (starman runs with umask 0, so a simple || was causing a failure)

Modified Paths

Modified: trunk/opsview-core/bin/create_and_send_configs
===================================================================
--- trunk/opsview-core/bin/create_and_send_configs	2012-07-26 15:55:25 UTC (rev 9625)
+++ trunk/opsview-core/bin/create_and_send_configs	2012-07-26 16:00:27 UTC (rev 9626)
@@ -132,7 +132,7 @@
 
 $Fork->schedule(
     run_on_start => sub {
-        umask 077 || die "umask failed";
+        die "Umask failed" unless defined umask 077;
         chdir "$host_dir" or die "failed to chdir $host_dir";
         my $rc = system(
             "tar --gzip -hcf $configs_tmp_dir/$configs_tar_gz $transfer_files $transfer_files_conf_d $master_files 2>&1"

_______________________________________________
Opsview-checkins mailing list
[email protected]
http://lists.opsview.org/lists/listinfo/opsview-checkins

Reply via email to