[Box Backup-commit] #59: Inconsistent treatment of NotifyScript between Box Backup and Boxi

[Box Backup]

#59: Inconsistent treatment of NotifyScript between Box Backup and Boxi
----------------------+-----------------------------------------------------
 Reporter:  achim     |       Owner:  ben    
     Type:  defect    |      Status:  new    
 Priority:  normal    |   Milestone:         
Component:  bbackupd  |     Version:  0.11rc2
 Keywords:            |  
----------------------+-----------------------------------------------------

Comment(by achim):

 Additional quotation marks work here:

 {{{
         // Script to run
         std::string script("\"\"" + conf.GetKeyValue("NotifyScript") + "\"
 " +
                         sEventNames[Event] + " \"" + GetConfigFileName() +
 "\"\"");

       // Then do it
       int returnCode = ::system(script.c_str());
       if(returnCode != 0)
 }}}

 but not in this case:
 {{{
         std::string script("\"\"" + conf.GetKeyValue("SyncAllowScript") +
 "\" " +
                 " \"" + GetConfigFileName() + "\"\"");


         // Run it?
         pid_t pid = 0;
         try
         {
                 std::auto_ptr<IOStream> pscript(LocalProcessStream(script,
                         pid));
 }}}

 resulting error for the latter is:

 {{{
 ERROR:   Failed to CreateProcess: '""d:\install\Online
 Backup\SyncAllowScript.cmd"  "C:\Documents and Settings\achimlatz\Da
 tos de programa\Online Backup\bbackupd.conf""': El sistema no puede hallar
 el archivo especificado. (2)
 WARNING: Exception thrown: ServerException(ServerForkError) at
 LocalProcessStream.cpp(223)
 ERROR:   Internal error running SyncAllowScript: Server ServerForkError
 (""d:\install\Online Backup\SyncAllowScript.cmd"  "
 C:\Documents and Settings\achimlatz\Datos de programa\Online
 Backup\bbackupd.conf"")
 }}}

-- 
Ticket URL: <https://www.boxbackup.org/trac/ticket/59#comment:2>
Box Backup <http://www.boxbackup.org/>
An open source, completely automatic on-line backup system for UNIX.