Merge lp://qastaging/~3v1n0/bamf/view-children-avoid-reload into lp://qastaging/bamf
Proposed by
Marco Trevisan (Treviño)
Status: | Merged |
---|---|
Approved by: | Andrea Azzarone |
Approved revision: | 572 |
Merged at revision: | 551 |
Proposed branch: | lp://qastaging/~3v1n0/bamf/view-children-avoid-reload |
Merge into: | lp://qastaging/bamf |
Diff against target: |
48 lines (+7/-2) 1 file modified
lib/libbamf/bamf-view.c (+7/-2) |
To merge this branch: | bzr merge lp://qastaging/~3v1n0/bamf/view-children-avoid-reload |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
Andrea Azzarone (community) | Approve | ||
Review via email: mp+169258@code.qastaging.launchpad.net |
Commit message
BamfView: don't try to reload the children if they're empty but we already tried once
This finally mutes the "Unable to fetch children" warning... \o/
Description of the change
If the children list of an bamf-view is empty because we've just removed the last children on a "children-removed" signal, we shouldn't try to ask again to the deaemon if it is really the case, we can trust to the signal and avoid to do further requests until the daemon lives.
To post a comment you must log in.
PASSED: Continuous integration, rev:572 jenkins. qa.ubuntu. com/job/ bamf-ci/ 52/ jenkins. qa.ubuntu. com/job/ bamf-saucy- amd64-ci/ 33 jenkins. qa.ubuntu. com/job/ bamf-saucy- armhf-ci/ 33
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild: s-jenkins: 8080/job/ bamf-ci/ 52/rebuild
http://