Application Indicators

Merge lp://qastaging/~ted/indicator-application/upstart-job into lp://qastaging/indicator-application/14.04

  1. upstart-job
  2. Merge into trunk.14.04
Proposed by Ted Gould
Status: Merged
Approved by: Charles Kerr
Approved revision: 252
Merged at revision: 242
Proposed branch: lp://qastaging/~ted/indicator-application/upstart-job
Merge into: lp://qastaging/indicator-application/14.04
Diff against target: 149 lines (+70/-20)
7 files modified
.bzrignore (+2/-0)
configure.ac (+0/-11)
data/Makefile.am (+13/-6)
data/indicator-application.conf.in (+15/-0)
data/indicator-application.desktop.in (+9/-0)
data/indicator-application.service.in (+0/-3)
src/application-service-watcher.c (+31/-0)
To merge this branch: bzr merge lp://qastaging/~ted/indicator-application/upstart-job
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Charles Kerr (community) Approve
Review via email: mp+192975@code.qastaging.launchpad.net

This proposal supersedes a proposal from 2013-05-29.

Description of the change

Switch indicator application over to having an upstart job for managing the service. Also make it emit it's own event for when application indicators should be started so we can clean up the boot sequence.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal

PASSED: Continuous integration, rev:249
http://jenkins.qa.ubuntu.com/job/indicator-application-ci/7/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-application-saucy-amd64-ci/1
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-application-saucy-armhf-ci/1
        deb: http://jenkins.qa.ubuntu.com/job/indicator-application-saucy-armhf-ci/1/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins:8080/job/indicator-application-ci/7/rebuild

review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal

PASSED: Continuous integration, rev:250
http://jenkins.qa.ubuntu.com/job/indicator-application-ci/8/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-application-saucy-amd64-ci/2
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-application-saucy-armhf-ci/2
        deb: http://jenkins.qa.ubuntu.com/job/indicator-application-saucy-armhf-ci/2/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins:8080/job/indicator-application-ci/8/rebuild

review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:252
http://jenkins.qa.ubuntu.com/job/indicator-application-ci/11/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-application-trusty-amd64-ci/3
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-application-trusty-armhf-ci/3
        deb: http://jenkins.qa.ubuntu.com/job/indicator-application-trusty-armhf-ci/3/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/indicator-application-ci/11/rebuild

review: Approve (continuous-integration)
Revision history for this message
Charles Kerr (charlesk) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

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