diff options
author | Jim Meyering <meyering@fb.com> | 2016-10-04 19:20:46 -0700 |
---|---|---|
committer | Jim Meyering <meyering@fb.com> | 2016-10-15 08:41:42 -0700 |
commit | dad7ab0b7b322a800e6b1012b777169169068388 (patch) | |
tree | 8ea9b6caf6a6e4d5e7ae645501d0a9f082d05cc7 /src/install.c | |
parent | 0d9323ec551ae3de965167247c32dc4d6a407862 (diff) | |
download | coreutils-dad7ab0b7b322a800e6b1012b777169169068388.tar.xz |
build: add die.h; avoid new warnings from GCC 7
* src/die.h (die): New file/function from grep.
Note: we expect this file to migrate to gnulib.
* src/csplit.c: Include die.h.
(check_format_conv_type): Use die in place of error-nonzero;break;
* src/install.c (strip): Likewise.
* src/nl.c (proc_text): Likewise. This also suppresses a new warning
from GCC 7's -Werror=strict-overflow.
* src/tail.c (parse_options): Likewise.
* src/basename.c (main): Adjust "fall through" comment
so that GCC 7's -Wimplicit-fallthrough honors it.
* src/cp.c (main): Add a "fall through" comment.
* src/ls.c (gobble_file): Likewise.
(get_funky_string): Adjust a "fall through" comment so it is
recognized.
* cfg.mk (exclude_file_name_regexp--sc_system_h_headers): Add die.h
to this list of exempt src/*.h files.
Diffstat (limited to 'src/install.c')
-rw-r--r-- | src/install.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/install.c b/src/install.c index 1e1fed5b6..8f512d8c6 100644 --- a/src/install.c +++ b/src/install.c @@ -31,6 +31,7 @@ #include "error.h" #include "cp-hash.h" #include "copy.h" +#include "die.h" #include "filenamecat.h" #include "full-read.h" #include "mkancesdirs.h" @@ -555,8 +556,7 @@ strip (char const *name) break; case 0: /* Child. */ execlp (strip_program, strip_program, name, NULL); - error (EXIT_FAILURE, errno, _("cannot run %s"), quoteaf (strip_program)); - break; + die (EXIT_FAILURE, errno, _("cannot run %s"), quoteaf (strip_program)); default: /* Parent. */ if (waitpid (pid, &status, 0) < 0) error (0, errno, _("waiting for strip")); |