1 files changed, 35 insertions, 10 deletions

diff --git a/miscutils/make.c b/miscutils/make.c
index d554a5f7f..8feec1202 100644
--- a/miscutils/make.c
+++ b/miscutils/make.c
@@ -293,6 +293,19 @@ vwarning(FILE *stream, const char *msg, va_list list)
 }
 
 /*
+ * Diagnostic handler.  Print message to standard error.
+ */
+static void
+diagnostic(const char *msg, ...)
+{
+        va_list list;
+
+        va_start(list, msg);
+        vwarning(stderr, msg, list);
+        va_end(list);
+}
+
+/*
  * Error handler.  Print message and exit.
  */
 static void error(const char *msg, ...) NORETURN;
@@ -2111,7 +2124,7 @@ remove_target(void)
         if (!dryrun && !print && !precious &&
                         target && !(target->n_flag & (N_PRECIOUS | N_PHONY)) &&
                         unlink(target->n_name) == 0) {
-                warning("'%s' removed", target->n_name);
+                diagnostic("'%s' removed", target->n_name);
         }
 }
 
@@ -2164,23 +2177,35 @@ docmds(struct name *np, struct cmd *cp)
 
                         target = np;
                         status = system(cmd);
-                        target = NULL;
                         // If this command was being run to create an include file
                         // or bring it up-to-date errors should be ignored and a
                         // failure status returned.
                         if (status == -1 && !doinclude) {
                                 error("couldn't execute '%s'", q);
                         } else if (status != 0 && !signore) {
-                                if (!doinclude)
-                                        warning("failed to build '%s'", np->n_name);
-#if !ENABLE_PLATFORM_MINGW32
-                                if (status == SIGINT || status == SIGQUIT)
-#endif
+                                if (!posix && WIFSIGNALED(status))
                                         remove_target();
-                                if (errcont || doinclude)
+                                if (errcont || doinclude) {
+                                        warning("failed to build '%s'", np->n_name);
                                         estat |= MAKE_FAILURE;
-                                else
-                                        exit(status);
+                                } else {
+                                        const char *err_type = NULL;
+                                        int err_value;
+
+                                        if (WIFEXITED(status)) {
+                                                err_type = "exit";
+                                                err_value = WEXITSTATUS(status);
+                                        } else if (WIFSIGNALED(status)) {
+                                                err_type = "signal";
+                                                err_value = WTERMSIG(status);
+                                        }
+
+                                        if (err_type)
+                                                error("failed to build '%s' %s %d", np->n_name,
+                                                                err_type, err_value);
+                                        else
+                                                error("failed to build '%s'", np->n_name);
+                                }
                         }
                 }
                 if (sdomake || dryrun || dotouch)