svn commit: trunk/busybox: debianutils libbb procps testsuite

[vda at busybox.net]

Author: vda
Date: 2007-11-24 20:54:13 -0800 (Sat, 24 Nov 2007)
New Revision: 20529

Log:
which: fix testsuite failure
pidof: do not output empty line if nothing found
testsuite: disable false positive



Modified:
   trunk/busybox/debianutils/which.c
   trunk/busybox/libbb/find_pid_by_name.c
   trunk/busybox/procps/pidof.c
   trunk/busybox/testsuite/pidof.tests
   trunk/busybox/testsuite/runtest


Changeset:
Modified: trunk/busybox/debianutils/which.c
===================================================================
--- trunk/busybox/debianutils/which.c	2007-11-25 03:15:24 UTC (rev 20528)
+++ trunk/busybox/debianutils/which.c	2007-11-25 04:54:13 UTC (rev 20529)
@@ -22,11 +22,11 @@
 		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++;

Modified: trunk/busybox/libbb/find_pid_by_name.c
===================================================================
--- trunk/busybox/libbb/find_pid_by_name.c	2007-11-25 03:15:24 UTC (rev 20528)
+++ trunk/busybox/libbb/find_pid_by_name.c	2007-11-25 04:54:13 UTC (rev 20529)
@@ -63,7 +63,7 @@
 		    (!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;

Modified: trunk/busybox/procps/pidof.c
===================================================================
--- trunk/busybox/procps/pidof.c	2007-11-25 03:15:24 UTC (rev 20528)
+++ trunk/busybox/procps/pidof.c	2007-11-25 04:54:13 UTC (rev 20529)
@@ -47,12 +47,13 @@
 	}
 #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 @@
 #endif
 		}
 		free(pidList);
-		optind++;
+		argv++;
 	}
-	bb_putchar('\n');
+	if (!first)
+		bb_putchar('\n');
 
 #if ENABLE_FEATURE_PIDOF_OMIT
 	if (ENABLE_FEATURE_CLEAN_UP)

Modified: trunk/busybox/testsuite/pidof.tests
===================================================================
--- trunk/busybox/testsuite/pidof.tests	2007-11-25 03:15:24 UTC (rev 20528)
+++ trunk/busybox/testsuite/pidof.tests	2007-11-25 04:54:13 UTC (rev 20529)
@@ -22,7 +22,9 @@
 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" "" "" ""
 

Modified: trunk/busybox/testsuite/runtest
===================================================================
--- trunk/busybox/testsuite/runtest	2007-11-25 03:15:24 UTC (rev 20528)
+++ trunk/busybox/testsuite/runtest	2007-11-25 04:54:13 UTC (rev 20529)
@@ -31,6 +31,7 @@
 	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