Dmitrijs Ledkovs has proposed merging 
lp:~xnox/upstart/clean-up-processes-reload-action into lp:upstart.

Requested reviews:
  Upstart Reviewers (upstart-reviewers)

For more details, see:
https://code.launchpad.net/~xnox/upstart/clean-up-processes-reload-action/+merge/181792
-- 
https://code.launchpad.net/~xnox/upstart/clean-up-processes-reload-action/+merge/181792
Your team Upstart Reviewers is requested to review the proposed merge of 
lp:~xnox/upstart/clean-up-processes-reload-action into lp:upstart.
=== modified file 'util/tests/test_initctl.c'
--- util/tests/test_initctl.c	2013-08-23 09:22:32 +0000
+++ util/tests/test_initctl.c	2013-08-23 11:26:33 +0000
@@ -8462,7 +8462,6 @@
 	FILE *          output;
 	FILE *          errors;
 	pid_t           server_pid;
-	pid_t           proc_pid;
 	DBusMessage *   method_call;
 	DBusMessage *   reply = NULL;
 	const char *    name_value;
@@ -8502,10 +8501,6 @@
 	 */
 	TEST_FEATURE ("with single argument");
 	TEST_ALLOC_FAIL {
-		TEST_CHILD (proc_pid) {
-			pause ();
-		}
-
 		TEST_CHILD (server_pid) {
 			/* Expect the GetJobByName method call on the
 			 * manager object, make sure the job name is passed
@@ -8635,9 +8630,6 @@
 
 			kill (server_pid, SIGTERM);
 			waitpid (server_pid, NULL, 0);
-
-			kill (proc_pid, SIGTERM);
-			waitpid (proc_pid, NULL, 0);
 			continue;
 		}
 
@@ -8660,10 +8652,6 @@
 	 */
 	TEST_FEATURE ("with multiple arguments");
 	TEST_ALLOC_FAIL {
-		TEST_CHILD (proc_pid) {
-			pause ();
-		}
-
 		TEST_CHILD (server_pid) {
 			/* Expect the GetJobByName method call on the
 			 * manager object, make sure the job name is passed
@@ -8797,9 +8785,6 @@
 
 			kill (server_pid, SIGTERM);
 			waitpid (server_pid, NULL, 0);
-
-			kill (proc_pid, SIGTERM);
-			waitpid (proc_pid, NULL, 0);
 			continue;
 		}
 
@@ -8827,10 +8812,6 @@
 	setenv ("UPSTART_INSTANCE", "foo", TRUE);
 
 	TEST_ALLOC_FAIL {
-		TEST_CHILD (proc_pid) {
-			pause ();
-		}
-
 		TEST_CHILD (server_pid) {
 			/* Expect the GetJobByName method call on the
 			 * manager object, make sure the job name is passed
@@ -8958,9 +8939,6 @@
 
 			kill (server_pid, SIGTERM);
 			waitpid (server_pid, NULL, 0);
-
-			kill (proc_pid, SIGTERM);
-			waitpid (proc_pid, NULL, 0);
 			continue;
 		}
 

-- 
upstart-devel mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/upstart-devel

Reply via email to