Marco Trevisan (Treviño) has proposed merging 
lp:~3v1n0/upstart/activation-proper-dbus-path into lp:upstart.

Commit message:
Control: use the /org/freedesktop/DBus canonical path for updating the 
activation environment variables

Requested reviews:
  Upstart Reviewers (upstart-reviewers)
Related bugs:
  Bug #1578934 in upstart (Ubuntu): " Method 'UpdateActivationEnvironment' is 
only available at the canonical object path '/org/freedesktop/DBus'"
  https://bugs.launchpad.net/ubuntu/+source/upstart/+bug/1578934

For more details, see:
https://code.launchpad.net/~3v1n0/upstart/activation-proper-dbus-path/+merge/294025
-- 
Your team Upstart Reviewers is requested to review the proposed merge of 
lp:~3v1n0/upstart/activation-proper-dbus-path into lp:upstart.
=== modified file 'init/control.c'
--- init/control.c	2015-05-16 19:12:37 +0000
+++ init/control.c	2016-05-06 16:06:27 +0000
@@ -1445,7 +1445,9 @@
 					dbus_vars[0]->item1 = nih_strdup (dbus_vars[0], "");
 				}
 
-				dbus_proxy = NIH_SHOULD (nih_dbus_proxy_new (dbus_vars, control_bus, "org.freedesktop.DBus", "/", NULL, NULL));
+				dbus_proxy = NIH_SHOULD (nih_dbus_proxy_new (dbus_vars, control_bus,
+					"org.freedesktop.DBus", "/org/freedesktop/DBus", NULL, NULL));
+
 				if (! dbus_proxy) {
 					nih_warn (_("Failed to get dbus_proxy"));
 				} else {

-- 
upstart-devel mailing list
upstart-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/upstart-devel

Reply via email to