[Merge] lp:~stgraber/upstart/upstart-make-event-bridge-usable into lp:upstart
Stéphane Graber
stgraber at stgraber.org
Tue Jan 22 21:54:21 UTC 2013
The proposal to merge lp:~stgraber/upstart/upstart-make-event-bridge-usable into lp:upstart has been updated.
Description changed to:
This branch addresses a few problems with the current implementation of the
event bridge.
== PID file handling ==
In the past, when started with --daemon, nih would try to create a pidfile
under /run, which would obviously fail when running as a user.
The first change is to instead use XDG_RUNTIME_DIR or HOME as the preferred
location for the pidfile.
This brings us to the second problem with the pidfile. A single user may have
multiple upstart instances running at the same time, each of those would
require its own event bridge. So the pidfile can't be unique to the user.
This one was fixed in two ways:
- upstart-event-bridge.conf no longer uses --daemon, so by default no pidfile
will be created at all.
- When called with --daemon, the pidfile now includes the PID of the upstart
instance, taken from the value of UPSTART_SESSION (last field).
== DBus socket ==
When the event bridge was initially written, we didn't have a private socket
for the user instance, so the bridge was connecting to the DBus session bus.
However, we know have that private socket and no longer use the session bus by
default.
So this branch now changes the default bus from the session bus to the upstart
user session private bus.
For more details, see:
https://code.launchpad.net/~stgraber/upstart/upstart-make-event-bridge-usable/+merge/144399
--
https://code.launchpad.net/~stgraber/upstart/upstart-make-event-bridge-usable/+merge/144399
Your team Upstart Reviewers is requested to review the proposed merge of lp:~stgraber/upstart/upstart-make-event-bridge-usable into lp:upstart.
More information about the upstart-devel
mailing list