Application Indicators

Merge lp://qastaging/~ted/indicator-application/systemd-unit into lp://qastaging/indicator-application/16.10

  1. systemd-unit
  2. Merge into trunk.16.10
Proposed by Ted Gould
Status: Merged
Approved by: Martin Pitt
Approved revision: 259
Merged at revision: 255
Proposed branch: lp://qastaging/~ted/indicator-application/systemd-unit
Merge into: lp://qastaging/indicator-application/16.10
Diff against target: 87 lines (+28/-1)
5 files modified
configure.ac (+4/-0)
data/Makefile.am (+13/-1)
data/indicator-application.override (+1/-0)
data/indicator-application.service.in (+8/-0)
debian/control (+2/-0)
To merge this branch: bzr merge lp://qastaging/~ted/indicator-application/systemd-unit
Reviewer Review Type Date Requested Status
Martin Pitt Approve
PS Jenkins bot continuous-integration Pending
Review via email: mp+300336@code.qastaging.launchpad.net

Commit message

Adding a systemd user unit

Description of the change

We're gonna have a systemd user session!

To post a comment you must log in.
lp://qastaging/~ted/indicator-application/systemd-unit updated
256. By Ted Gould

Adding an after clause to ensure they start before us

257. By Ted Gould

Restructuring our dependencies

258. By Ted Gould

Adding systemd as a build depend and indicator-common as a runtime one

259. By Ted Gould

Adding a restart rule

Revision history for this message
Martin Pitt (pitti) wrote :

LGTM, and works well, thank you!

review: Approve

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