Steve Langasek has proposed merging lp:~vorlon/upstart/lp.1235480 into 
lp:upstart.

Requested reviews:
  Upstart Reviewers (upstart-reviewers)
Related bugs:
  Bug #1235480 in upstart : "upstart-local-bridge clobbers $PATH with its 
events"
  https://bugs.launchpad.net/upstart/+bug/1235480

For more details, see:
https://code.launchpad.net/~vorlon/upstart/lp.1235480/+merge/189434

just as for the dbus bridge, we need the local bridge to not clobber $PATH.
-- 
https://code.launchpad.net/~vorlon/upstart/lp.1235480/+merge/189434
Your team Upstart Reviewers is requested to review the proposed merge of 
lp:~vorlon/upstart/lp.1235480 into lp:upstart.
=== modified file 'ChangeLog'
--- ChangeLog	2013-10-04 12:33:07 +0000
+++ ChangeLog	2013-10-04 21:36:48 +0000
@@ -1,3 +1,8 @@
+2013-01-04  Steve Langasek  <[email protected]
+
+	* extra/upstart-local-bridge.c: use SOCKET_PATH in our event
+	  environment, instead of clobbering PATH.  (LP: #1235480)
+
 2013-10-04  James Hunt  <[email protected]>
 
 	* extra/upstart-local-bridge.c:

=== modified file 'extra/upstart-local-bridge.c'
--- extra/upstart-local-bridge.c	2013-10-04 11:12:44 +0000
+++ extra/upstart-local-bridge.c	2013-10-04 21:36:48 +0000
@@ -824,7 +824,7 @@
 	var = NIH_MUST (nih_sprintf (NULL, "CLIENT_PID=%u", (unsigned int)client->ucred.pid));
 	NIH_MUST (nih_str_array_addp (&env, NULL, NULL, var));
 
-	var = NIH_MUST (nih_sprintf (NULL, "PATH=%s", socket_path));
+	var = NIH_MUST (nih_sprintf (NULL, "SOCKET_PATH=%s", socket_path));
 	NIH_MUST (nih_str_array_addp (&env, NULL, NULL, var));
 
 	/* Add the name=value pair */

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

Reply via email to