X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fmisc.cpp;h=aadc056ea334f3784ad5adf124ca5919c465239d;hb=0458300fda4f345f865a7f3ee4fc0f2020a91983;hp=a299f0e54b6f1f36c05d338d8c5397eef36495e8;hpb=bdc8b6638b486aa668b4a9c6c5cce5f6b5f18222;p=builder.git

diff --git a/source/misc.cpp b/source/misc.cpp
index a299f0e..aadc056 100644
--- a/source/misc.cpp
+++ b/source/misc.cpp
@@ -1,64 +1,75 @@
 /* $Id$
 
 This file is part of builder
-Copyright © 2006-2007 Mikko Rasa, Mikkosoft Productions
+Copyright © 2006-2009  Mikko Rasa, Mikkosoft Productions
 Distributed under the LGPL
 */
 
-#include <iostream>
 #include <sys/wait.h>
+#include <fcntl.h>
 #include <cstdlib>
 #include <cstring>
+#include <msp/io/print.h>
 #include "misc.h"
 
 using namespace std;
 using namespace Msp;
 
-/**
-Runs a command and returns its output as a string.  The exit status of the
-command is lost.
-*/
-string run_command(const StringList &argv)
+string run_command(const StringList &argv, int *status)
 {
 	int pfd[2];
 	pipe(pfd);
 
 	string result;
 
-	pid_t pid=fork();
+	pid_t pid = fork();
 	if(pid==0)
 	{
 		char *argv_[argv.size()+1];
 
-		unsigned j=0;
+		unsigned j = 0;
 		for(StringList::const_iterator i=argv.begin(); i!=argv.end(); ++i)
-			argv_[j++]=strdup(i->c_str());
-		argv_[j]=0;
+			argv_[j++] = strdup(i->c_str());
+		argv_[j] = 0;
 
 		close(pfd[0]);
 		dup2(pfd[1], 1);
-		dup2(pfd[1], 2);
+		close(pfd[1]);
+		int devnull = open("/dev/null", O_WRONLY);
+		dup2(devnull, 2);
+		close(devnull);
 
 		execvp(argv_[0], argv_);
 		::exit(1);
 	}
 	else if(pid==-1)
-		cerr<<"Failed to execute "<<argv.front()<<'\n';
+		IO::print(IO::cerr, "Failed to execute %s\n", argv.front());
 	else
 	{
 		close(pfd[1]);
 		while(1)
 		{
 			char buf[1024];
-			int len=read(pfd[0], buf, sizeof(buf));
+			int len = read(pfd[0], buf, sizeof(buf));
 			if(len<=0)
 			{
-				if(waitpid(pid, 0, WNOHANG))
+				int s;
+				if(waitpid(pid, &s, WNOHANG))
+				{
+					if(status)
+					{
+						if(WIFEXITED(s))
+							*status = WEXITSTATUS(s);
+						else
+							*status = -1;
+					}
 					break;
+				}
 			}
 			else
 				result.append(buf, len);
 		}
+		close(pfd[0]);
 	}
 
 	return result;