diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2014-09-18 02:47:05 +0400 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2014-09-18 02:47:05 +0400 |
commit | cd7a38a87d54f5421b379870ff866676f31923b2 (patch) | |
tree | ddc2a3e74dcbf1cd0486e01c4f0e6c48bb445988 | |
parent | d0cdacafa98ec0e73e58c2e5a8ba9dded18006df (diff) |
false: make "false --help" exit with 1
function old new delta
run_applet_no_and_exit 447 445 -2
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r-- | applets/applet_tables.c | 17 | ||||
-rw-r--r-- | libbb/appletlib.c | 26 |
2 files changed, 35 insertions, 8 deletions
diff --git a/applets/applet_tables.c b/applets/applet_tables.c index 94b974e09..92bf1e447 100644 --- a/applets/applet_tables.c +++ b/applets/applet_tables.c @@ -14,6 +14,7 @@ #include <string.h> #include <stdio.h> #include <unistd.h> +#include <ctype.h> #undef ARRAY_SIZE #define ARRAY_SIZE(x) ((unsigned)(sizeof(x) / sizeof((x)[0]))) @@ -49,6 +50,16 @@ static int cmp_name(const void *a, const void *b) return strcmp(aa->name, bb->name); } +static int str_isalnum_(const char *s) +{ + while (*s) { + if (!isalnum(*s) && *s != '_') + return 0; + s++; + } + return 1; +} + int main(int argc, char **argv) { int i; @@ -94,6 +105,12 @@ int main(int argc, char **argv) } printf(";\n\n"); + for (i = 0; i < NUM_APPLETS; i++) { + if (str_isalnum_(applets[i].name)) + printf("#define APPLET_NO_%s %d\n", applets[i].name, i); + } + printf("\n"); + printf("#ifndef SKIP_applet_main\n"); printf("int (*const applet_main[])(int argc, char **argv) = {\n"); for (i = 0; i < NUM_APPLETS; i++) { diff --git a/libbb/appletlib.c b/libbb/appletlib.c index a0150854a..cb16e310f 100644 --- a/libbb/appletlib.c +++ b/libbb/appletlib.c @@ -745,15 +745,25 @@ void FAST_FUNC run_applet_no_and_exit(int applet_no, char **argv) /* Reinit some shared global data */ xfunc_error_retval = EXIT_FAILURE; - applet_name = APPLET_NAME(applet_no); - if (argc == 2 && strcmp(argv[1], "--help") == 0) { - /* Special case. POSIX says "test --help" - * should be no different from e.g. "test --foo". */ -//TODO: just compare applet_no with APPLET_NO_test - if (!ENABLE_TEST || strcmp(applet_name, "test") != 0) { - /* If you want "foo --help" to return 0: */ - xfunc_error_retval = 0; + +#if defined APPLET_NO_test + /* Special case. POSIX says "test --help" + * should be no different from e.g. "test --foo". + * Thus for "test", we skip --help check. + */ + if (applet_no != APPLET_NO_test) +#endif + { + if (argc == 2 && strcmp(argv[1], "--help") == 0) { +#if defined APPLET_NO_false + /* Someone insisted that "false --help" must exit 1. Sigh */ + if (applet_no != APPLET_NO_false) +#endif + { + /* Make "foo --help" exit with 0: */ + xfunc_error_retval = 0; + } bb_show_usage(); } } |