Use set -e, -x in shell process, handle exit status correctly

This makes the default behavior like make(1), but it is configurable.
Furthermore, it is still a single shell process, which is arguably
a benefit.

1 files changed, 21 insertions, 13 deletions

diff --git a/build.c b/build.c
index fc21925..600ac92 100644
--- a/build.c
+++ b/build.c
@@ -4,6 +4,7 @@
 #include <stdio.h>
 #include <string.h>
 #include <sys/stat.h>
+#include <sys/wait.h>
 #include <unistd.h>
 
 #define USAGE "usage: %s file [...]\n", name
@@ -32,7 +33,7 @@ main(int argc, char *argv[])
 {
 	char *b, buf[MAXBUF], *cmd[MAXCMDS], *d, *dep, *name, *tgt;
 	FILE *fp;
-	int c, dflag, fflag, i, icmd, j, s, status;
+	int c, dflag, fflag, i, icmd, j, s;
 	struct stat sb, ssb;
 
 	/* allocate memory */
@@ -43,10 +44,9 @@ main(int argc, char *argv[])
 	dep = malloc(MAXDEP*sizeof(char));
 	if (!dep) err(1, "malloc");
 	tgt = malloc(MAXTGT*sizeof(char));
-	if (!dep) err(1, "malloc");
+	if (!tgt) err(1, "malloc");
 
 	tgt[0] = dep[0] = 0;
-	status = 0;
 
 	/* process arguments */
 	name = argv[0];
@@ -85,13 +85,13 @@ main(int argc, char *argv[])
 				/* command line */
 				if (strncmp(b, "	$ ", 3) == 0
 				 || strncmp(b, " $ ", 3) == 0) {
-					strncpy(cmd[icmd++],b+3,MAXBUF-1);
+					strncpy(cmd[icmd++], b+3, MAXBUF-1);
 
 					/* find target */
 					for (b = b+3; *b; b++) {
 						if (!(*b+1)) continue;
 						if (*b != '>') continue;
-						strncpy(tgt,b+1,MAXTGT-1);
+						strncpy(tgt, b+1, MAXTGT-1);
 
 					}
 
@@ -113,7 +113,7 @@ main(int argc, char *argv[])
 
 		if (!icmd) {
 			fprintf(stderr, "%s: no command line found\n", argv[i]);
-			continue;
+			goto next;
 		}
 
 		if (fflag || !*tgt)
@@ -153,25 +153,33 @@ main(int argc, char *argv[])
 
 uptodate:
 		fprintf(stderr, "%s: already up-to-date\n", argv[i]);
-		goto done;
+		goto next;
 
 build:
 		/* run commands */
 		buf[0] = 0;
+		strcat(buf, "set -ex\n");
 		for (j = 0; j < icmd; j++) {
-			fprintf(stderr, "%s: $ %s\n", argv[i], cmd[j]);
 			strncat(buf, cmd[j], MAXBUF-1);
 			strncat(buf, "\n", MAXBUF-1);
 		}
 		s = system(buf);
-		if (s != -1)
-			status += s;
+		if (s == 0)
+			goto next;
+
+		if (WIFEXITED(s)) {
+			fprintf(stderr, "%s: exited with %d\n",
+				argv[i], WEXITSTATUS(s));
+			exit(WEXITSTATUS(s));
+		} else if (WIFSIGNALED(s)) {
+			fprintf(stderr, "%s: terminated by signal %d\n",
+				argv[i], WTERMSIG(s));
+			exit(1);
+		}
 
-done:
+next:
 		fclose(fp);
 	}
-
-	return status > 0;
 }
 
 void