libindicator

Merge lp://qastaging/~charlesk/libindicator/lp-1218222 into lp://qastaging/libindicator/14.04

  1. lp-1218222
  2. Merge into trunk.14.04
Proposed by Charles Kerr
Status: Merged
Approved by: Lars Karlitski
Approved revision: 516
Merged at revision: 515
Proposed branch: lp://qastaging/~charlesk/libindicator/lp-1218222
Merge into: lp://qastaging/libindicator/14.04
Diff against target: 63 lines (+8/-17)
1 file modified
libindicator/indicator-service.c (+8/-17)
To merge this branch: bzr merge lp://qastaging/~charlesk/libindicator/lp-1218222
Reviewer Review Type Date Requested Status
Lars Karlitski (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+192450@code.qastaging.launchpad.net

Description of the change

When we can't get a handle to the bus, exit gracefully instead of crashing.

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

PASSED: Continuous integration, rev:516
http://jenkins.qa.ubuntu.com/job/libindicator-ci/48/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/libindicator-trusty-amd64-ci/1
    SUCCESS: http://jenkins.qa.ubuntu.com/job/libindicator-trusty-armhf-ci/1
        deb: http://jenkins.qa.ubuntu.com/job/libindicator-trusty-armhf-ci/1/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/libindicator-ci/48/rebuild

review: Approve (continuous-integration)
Revision history for this message
Lars Karlitski (larsu) wrote :

Hm, I think you should have left the OMG! in there ;)

Thanks!

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