Changeset 2658

Timestamp:

08/03/2010 21:30:10 (2 years ago)

Author:

chris

Message:

Merge part of [2597] from trunk into 0.11rc7. Revert the quotes part of
[2588] as it breaks the unit tests.

File:

• box/RELEASE/0.11rc7/bin/bbackupd/BackupDaemon.cpp (modified) (6 diffs)

box/RELEASE/0.11rc7/bin/bbackupd/BackupDaemon.cpp

@@ -1018 +1018 @@
         int waitInSeconds = (60*5);
 
+        std::string script(conf.GetKeyValue("SyncAllowScript") + 
+                " \"" + GetConfigFileName() + "\"");
+
         // Run it?
         pid_t pid = 0;
         try
         {
-                std::string script("\"" + conf.GetKeyValue("SyncAllowScript") + 
-                        "\" \"" + GetConfigFileName() + "\"");
-
-                std::auto_ptr<IOStream> pscript(LocalProcessStream(
-                        script.c_str(), pid));
+                std::auto_ptr<IOStream> pscript(LocalProcessStream(script,
+                        pid));
 
                 // Read in the result
@@ -1049 +1049 @@
                                 {
                                         BOX_ERROR("Invalid output from "
-                                                "SyncAllowScript " <<
-                                                script << ": '" << line << "'");
+                                                "SyncAllowScript: '" <<
+                                                line << "' (" << script << ")");
                                         throw;
                                 }
 
                                 BOX_NOTICE("Delaying sync by " << waitInSeconds
-                                        << " seconds (SyncAllowScript "
-                                        << conf.GetKeyValue("SyncAllowScript")
-                                        << ")");
+                                        << " seconds due to SyncAllowScript "
+                                        << "(" << script << ")");
                         }
                 }
@@ -1065 +1064 @@
         {
                 BOX_ERROR("Internal error running SyncAllowScript: "
-                        << e.what());
+                        << e.what() << " (" << script << ")");
         }
         catch(...)
@@ -1071 +1070 @@
                 // Ignore any exceptions
                 // Log that something bad happened
-                BOX_ERROR("Error running SyncAllowScript '"
-                        << conf.GetKeyValue("SyncAllowScript") << "'");
+                BOX_ERROR("Unknown error running SyncAllowScript (" <<
+                        script << ")");
         }
 
@@ -2189 +2188 @@
                 {
                         BOX_INFO("Not notifying administrator about event "
-                                << sEventNames[Event] << " -- set NotifyScript "
+                                << sEventNames[Event] << ", set NotifyScript "
                                 "to do this in future");
                 }
@@ -2196 +2195 @@
 
         // Script to run
-        std::string script("\"" + conf.GetKeyValue("NotifyScript") + "\" " +
+        std::string script(conf.GetKeyValue("NotifyScript") + " " +
                 sEventNames[Event] + " \"" + GetConfigFileName() + "\"");
         
         // Log what we're about to do
         BOX_INFO("About to notify administrator about event "
-                << sEventNames[Event] << ", running script " << script);
+                << sEventNames[Event] << ", running script '" << script << "'");
         
         // Then do it