Merge lp://staging/~mvo/software-center/lp772549 into lp://staging/software-center

Proposed by Michael Vogt
Status: Merged
Merged at revision: 2970
Proposed branch: lp://staging/~mvo/software-center/lp772549
Merge into: lp://staging/software-center
Diff against target: 78 lines (+33/-7)
1 file modified
softwarecenter/ui/gtk3/app.py (+33/-7)
To merge this branch: bzr merge lp://staging/~mvo/software-center/lp772549
Reviewer Review Type Date Requested Status
Gary Lasker (community) Approve
Review via email: mp+101869@code.staging.launchpad.net

Description of the change

This branch fixes at least one incarnation of #772549. It also adds some cleanup around the dbus
controller to ensure that its really stopped and working properly. Most of it is probably not needed,
but given that annoying effects if it fails the extra measures are probably a good idea.

To test:
1. open software-center
2. close it via window-close-button really quickly
3. open software-center again in a new terminal
4. see that it shows a empty window

(4) is fixed with this branch.

To post a comment you must log in.
2973. By Michael Vogt

softwarecenter/ui/gtk3/app.py: move the Gtk.main_quit() before the dbusController.stop and add exception handler for good measure

Revision history for this message
Gary Lasker (gary-lasker) wrote :

Michael, this is great! Does the trick for me perfectly for the case of the early quit of USC.

This is a really annoying bug and it's great to have it fixed, thank you!!

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