Indicator Date and Time

Merge lp://staging/~ted/indicator-datetime/systemd-unit into lp://staging/indicator-datetime

  1. systemd-unit
  2. Merge into trunk.16.10
Proposed by Ted Gould
Status: Merged
Approved by: Martin Pitt
Approved revision: 460
Merged at revision: 463
Proposed branch: lp://staging/~ted/indicator-datetime/systemd-unit
Merge into: lp://staging/indicator-datetime
Diff against target: 82 lines (+44/-0)
4 files modified
data/CMakeLists.txt (+32/-0)
data/indicator-datetime.override (+1/-0)
data/indicator-datetime.service.in (+8/-0)
debian/control (+3/-0)
To merge this branch: bzr merge lp://staging/~ted/indicator-datetime/systemd-unit
Reviewer Review Type Date Requested Status
Martin Pitt Approve
unity-api-1-bot continuous-integration Approve
PS Jenkins bot continuous-integration Pending
Indicator Applet Developers Pending
Review via email: mp+300420@code.staging.launchpad.net

Commit message

Adding a systemd user unit

Description of the change

New fun snappy stuff

To post a comment you must log in.
lp://staging/~ted/indicator-datetime/systemd-unit updated
458. By Ted Gould

Vivid is amazing

459. By Ted Gould

Adding a restart rule

460. By Ted Gould

Merging trunk and resolving conflicts

Revision history for this message
unity-api-1-bot (unity-api-1-bot) wrote :

PASSED: Continuous integration, rev:460
https://jenkins.canonical.com/unity-api-1/job/lp-indicator-datetime-ci/2/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/unity-api-1/job/build/605
    SUCCESS: https://jenkins.canonical.com/unity-api-1/job/build-0-fetch/611
    SUCCESS: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=amd64,release=vivid+overlay/434
        deb: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=amd64,release=vivid+overlay/434/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=amd64,release=xenial+overlay/434
        deb: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=amd64,release=xenial+overlay/434/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=amd64,release=yakkety/434
        deb: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=amd64,release=yakkety/434/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=armhf,release=vivid+overlay/434
        deb: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=armhf,release=vivid+overlay/434/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=armhf,release=xenial+overlay/434
        deb: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=armhf,release=xenial+overlay/434/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=armhf,release=yakkety/434
        deb: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=armhf,release=yakkety/434/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=i386,release=vivid+overlay/434
        deb: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=i386,release=vivid+overlay/434/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=i386,release=xenial+overlay/434
        deb: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=i386,release=xenial+overlay/434/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=i386,release=yakkety/434
        deb: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=i386,release=yakkety/434/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://jenkins.canonical.com/unity-api-1/job/lp-indicator-datetime-ci/2/rebuild

review: Approve (continuous-integration)
Revision history for this message
Martin Pitt (pitti) wrote :

This ends up as ExecStart=/indicator-datetime-service, so the unit fails to start.

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

Followup MP in https://code.launchpad.net/~pitti/indicator-datetime/fix-systemd-unit/+merge/308786, I added that to the ticket.

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