Merge lp://qastaging/~cmiller/desktopcouch/please-do-not-kill-process-group-because-dbus-hates-that into lp://qastaging/desktopcouch

Proposed by Chad Miller
Status: Merged
Approved by: Chad Miller
Approved revision: 148
Merged at revision: not available
Proposed branch: lp://qastaging/~cmiller/desktopcouch/please-do-not-kill-process-group-because-dbus-hates-that
Merge into: lp://qastaging/desktopcouch
Diff against target: 24 lines (+5/-4)
1 file modified
bin/desktopcouch-service (+5/-4)
To merge this branch: bzr merge lp://qastaging/~cmiller/desktopcouch/please-do-not-kill-process-group-because-dbus-hates-that
Reviewer Review Type Date Requested Status
Ubuntu One hackers Pending
Review via email: mp+23054@code.qastaging.launchpad.net

Commit message

My previous patch screws up the child-parent process condition, so desktopcouch-service tries to kill the process group. That's bad. DBus is in the process group, if something starts us up. Gnome really wants DBus to be alive.

Also, make the child replication service halfway up the possible niceness scale.

To post a comment you must log in.
Revision history for this message
Chad Miller (cmiller) wrote :

I'm unilaterally approving this. What I made before is badwrong.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches