diff options
| -rw-r--r-- | ChangeLog | 4 | ||||
| -rw-r--r-- | src/install.c | 9 |
2 files changed, 8 insertions, 5 deletions
@@ -1,5 +1,9 @@ 2007-03-17 Jim Meyering <jim@meyering.net> + Avoid an obscure build failure, prefer waitpid over wait. + * src/install.c (strip): Use waitpid, not wait. It's equivalent, + but feels less obsolescent. + * bootstrap: Don't use \> in grep regexp. For HP-UX. 2007-03-16 Jim Meyering <jim@meyering.net> diff --git a/src/install.c b/src/install.c index 8fc6a8a58..04577518c 100644 --- a/src/install.c +++ b/src/install.c @@ -566,11 +566,10 @@ strip (char const *name) error (EXIT_FAILURE, errno, _("cannot run strip")); break; default: /* Parent. */ - /* Parent process. */ - while (pid != wait (&status)) /* Wait for kid to finish. */ - /* Do nothing. */ ; - if (status) - error (EXIT_FAILURE, 0, _("strip failed")); + if (waitpid (pid, &status, 0) < 0) + error (EXIT_FAILURE, errno, _("waiting for strip")); + else if (! WIFEXITED (status) || WEXITSTATUS (status)) + error (EXIT_FAILURE, 0, _("strip process terminated abnormally")); break; } } |