sbr/execprog.c: Fix the return value
authormarkus schnalke <meillo@marmaro.de>
Thu, 12 Feb 2015 06:40:50 +0000 (07:40 +0100)
committermarkus schnalke <meillo@marmaro.de>
Thu, 12 Feb 2015 06:40:50 +0000 (07:40 +0100)
Use pidXwait(), which invokes pidstatus(), here for error reporting.
This eases the use of execprog() because we don't have to call it
there everytime. However, it might be unwanted to put the error
printing stuff into execprog() because we might have an invokation
of execprog() that want's to deal with errors itself in a different
way. If this would be the case, then we likely should change
pidXwait() to pidwait() and call pidstatus(execprog()).

sbr/execprog.c

index 9cc8879..ce38307 100644 (file)
@@ -27,7 +27,7 @@ execprog(char *cmd, char **arg)
 
        default:
                /* parent */
 
        default:
                /* parent */
-               return (pidwait(pid, -1) & 0377 ? 1 : 0);
+               return pidXwait(pid, cmd);
        }
 
        return 1;  /* NOT REACHED */
        }
 
        return 1;  /* NOT REACHED */