diff options
author | Denis Vlasenko <vda.linux@googlemail.com> | 2007-11-25 07:54:13 +0300 |
---|---|---|
committer | Denis Vlasenko <vda.linux@googlemail.com> | 2007-11-25 07:54:13 +0300 |
commit | 6446c2d4aba02eef0d8553c882f8d235288a66d9 (patch) | |
tree | 1637549e2c7e6432e6ec36774d0e5aa48e0bf44e | |
parent | 72e76044cfda377486a5199a0d35d71edf669a42 (diff) |
which: fix testsuite failure
pidof: do not output empty line if nothing found
testsuite: disable false positive
-rw-r--r-- | debianutils/which.c | 4 | ||||
-rw-r--r-- | libbb/find_pid_by_name.c | 2 | ||||
-rw-r--r-- | procps/pidof.c | 10 | ||||
-rwxr-xr-x | testsuite/pidof.tests | 4 | ||||
-rwxr-xr-x | testsuite/runtest | 1 |
5 files changed, 13 insertions, 8 deletions
diff --git a/debianutils/which.c b/debianutils/which.c index 883813059..5ab67194d 100644 --- a/debianutils/which.c +++ b/debianutils/which.c @@ -22,11 +22,11 @@ int which_main(int argc, char **argv) bb_show_usage(); } -/* We shouldn't do this. Ever. Not our business. + /* This matches what is seen on e.g. ubuntu + * "which" there is a shell script */ if (!getenv("PATH")) { putenv((char*)bb_PATH_root_path); } -*/ while (--argc > 0) { argv++; diff --git a/libbb/find_pid_by_name.c b/libbb/find_pid_by_name.c index 1fafec83a..8dcdb13bc 100644 --- a/libbb/find_pid_by_name.c +++ b/libbb/find_pid_by_name.c @@ -63,7 +63,7 @@ pid_t* find_pid_by_name(const char* procName) (!p->comm[sizeof(p->comm)-2] && strcmp(p->comm, procName) == 0) /* or we require argv0 to match (essential for matching reexeced /proc/self/exe)*/ || (p->argv0 && strcmp(bb_basename(p->argv0), procName) == 0) - /* TOOD: we can also try exe, do we want that? */ + /* TOOD: we can also try /proc/NUM/exe link, do we want that? */ ) { pidList = xrealloc(pidList, sizeof(*pidList) * (i+2)); pidList[i++] = p->pid; diff --git a/procps/pidof.c b/procps/pidof.c index f29561789..86d1957a6 100644 --- a/procps/pidof.c +++ b/procps/pidof.c @@ -47,12 +47,13 @@ int pidof_main(int argc, char **argv) } #endif /* Looks like everything is set to go. */ - while (optind < argc) { + argv += optind; + while (*argv) { pid_t *pidList; pid_t *pl; /* reverse the pidlist like GNU pidof does. */ - pidList = pidlist_reverse(find_pid_by_name(argv[optind])); + pidList = pidlist_reverse(find_pid_by_name(*argv)); for (pl = pidList; *pl; pl++) { #if ENABLE_FEATURE_PIDOF_OMIT if (opt & OPT_OMIT) { @@ -74,9 +75,10 @@ int pidof_main(int argc, char **argv) #endif } free(pidList); - optind++; + argv++; } - bb_putchar('\n'); + if (!first) + bb_putchar('\n'); #if ENABLE_FEATURE_PIDOF_OMIT if (ENABLE_FEATURE_CLEAN_UP) diff --git a/testsuite/pidof.tests b/testsuite/pidof.tests index bfde26e2c..29cfa943a 100755 --- a/testsuite/pidof.tests +++ b/testsuite/pidof.tests @@ -22,7 +22,9 @@ optional FEATURE_PIDOF_SINGLE testing "pidof -s" "pidof -s init" "1\n" "" "" optional FEATURE_PIDOF_OMIT -testing "pidof -o %PPID" "pidof -o %PPID pidof.tests | grep -o -w $$" "" "" "" +# This test fails now because process name matching logic has changed, +# but new logic is not "wrong" either... see find_pid_by_name.c comments +#testing "pidof -o %PPID" "pidof -o %PPID pidof.tests | grep -o -w $$" "" "" "" testing "pidof -o %PPID NOP" "pidof -o %PPID -s init" "1\n" "" "" testing "pidof -o init" "pidof -o 1 init | grep -o -w 1" "" "" "" diff --git a/testsuite/runtest b/testsuite/runtest index 4946cbd0d..0f081f9a5 100755 --- a/testsuite/runtest +++ b/testsuite/runtest @@ -31,6 +31,7 @@ run_applet_testcase() mkdir -p tmp pushd tmp > /dev/null +# echo Running testcase $testcase d=$tsdir sh -x -e $testcase >.logfile.txt 2>&1 || status=$? if [ $status -ne 0 ]; then |