Merge lp:~charlesk/indicator-sound/lp-1432446-use-bus-watcher-for-notifications into lp:indicator-sound/15.04
Proposed by
Charles Kerr
Status: | Merged |
---|---|
Approved by: | dobey |
Approved revision: | 493 |
Merged at revision: | 495 |
Proposed branch: | lp:~charlesk/indicator-sound/lp-1432446-use-bus-watcher-for-notifications |
Merge into: | lp:indicator-sound/15.04 |
Diff against target: |
34 lines (+4/-13) 1 file modified
src/service.vala (+4/-13) |
To merge this branch: | bzr merge lp:~charlesk/indicator-sound/lp-1432446-use-bus-watcher-for-notifications |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
dobey (community) | Approve | ||
Review via email: mp+256861@code.launchpad.net |
Commit message
Use the BusWatcher to look for org.freedesktop
Description of the change
Use BusWatcher, rather than a GDBusProxy, to watch for org.freedesktop
This crash is happening when there are bigger problems on the system, evidenced by the ticket's description of a blank white screen before this crash shows up. i-sound isn't much use without a working session bus, but at least we can stop it from generating redundant bug reports.
To post a comment you must log in.
PASSED: Continuous integration, rev:493 jenkins. qa.ubuntu. com/job/ indicator- sound-ci/ 224/ jenkins. qa.ubuntu. com/job/ indicator- sound-vivid- amd64-ci/ 51 jenkins. qa.ubuntu. com/job/ indicator- sound-vivid- armhf-ci/ 51 jenkins. qa.ubuntu. com/job/ indicator- sound-vivid- armhf-ci/ 51/artifact/ work/output/ *zip*/output. zip
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/indicator- sound-ci/ 224/rebuild
http://