]> git.tdb.fi Git - builder.git/blobdiff - source/externalaction.cpp
Replace per-file copyright notices with a single file
[builder.git] / source / externalaction.cpp
index 0ae68ff05f30bd297b314de7a77ac3e471192cb0..cad9ba8b23d659e3be069ce1dd7eee57c0c4e5df 100644 (file)
@@ -1,12 +1,45 @@
+#include <cstring>
+#include <cstdlib>
 #include <sys/wait.h>
-#include <iostream>
-#include <msp/iter.h>
+#include <msp/fs/dir.h>
+#include <msp/io/print.h>
+#include <msp/strings/utils.h>
 #include "builder.h"
 #include "externalaction.h"
 
 using namespace std;
 using namespace Msp;
 
+void ExternalAction::launch()
+{
+       if(builder.get_verbose()>=2)
+               IO::print("%s\n", join(argv.begin(), argv.end()));
+
+       if(builder.get_dry_run())
+               pid = -1;
+       else
+       {
+               pid = fork();
+               if(pid==0)
+               {
+                       char *argv_[argv.size()+1];
+
+                       unsigned j = 0;
+                       for(StringList::iterator i=argv.begin(); i!=argv.end(); ++i)
+                               argv_[j++] = strdup(i->c_str());
+                       argv_[j] = 0;
+
+                       if(!work_dir.empty())
+                               FS::chdir(work_dir);
+                       execvp(argv_[0], argv_);
+                       IO::print("Couldn't execute %s\n", argv.front());
+                       exit(1);
+               }
+               else if(pid<0)
+                       pid = 0;
+       }
+}
+
 int ExternalAction::check()
 {
        if(builder.get_dry_run())
@@ -14,7 +47,7 @@ int ExternalAction::check()
                signal_done.emit();
                return 0;
        }
-       
+
        if(!pid)
                return exit_code;
 
@@ -23,47 +56,12 @@ int ExternalAction::check()
        {
                signal_done.emit();
                if(WIFEXITED(status))
-                       exit_code=WEXITSTATUS(status);
+                       exit_code = WEXITSTATUS(status);
                else
-                       exit_code=254;
-               pid=0;
+                       exit_code = 254;
+               pid = 0;
                return exit_code;
        }
        else
                return -1;
 }
-
-/**
-Starts the external program.  Fill in argv before calling this.
-*/
-void ExternalAction::launch()
-{
-       if(builder.get_verbose()>=2)
-       {
-               for(StringList::const_iterator i=argv.begin(); i!=argv.end(); ++i)
-               {
-                       if(i!=argv.begin())
-                               cout<<' ';
-                       cout<<*i;
-               }
-               cout<<'\n';
-       }
-       
-       if(builder.get_dry_run())
-               pid=-1;
-       else
-       {
-               pid=fork();
-               if(pid==0)
-               {
-                       char *argv_[argv.size()+1];
-                       for(CountingIterator<string, StringList::iterator> i=argv.begin(); i!=argv.end(); ++i)
-                               argv_[i.count()]=strdup(i->c_str());
-                       argv_[argv.size()]=0;
-                       execvp(argv_[0], argv_);
-                       exit(1);
-               }
-               else if(pid<0)
-                       pid=0;
-       }
-}