Merge lp://qastaging/~mvo/software-center/lp914393 into lp://qastaging/software-center
Proposed by
Michael Vogt
Status: | Merged |
---|---|
Merged at revision: | 2971 |
Proposed branch: | lp://qastaging/~mvo/software-center/lp914393 |
Merge into: | lp://qastaging/software-center |
Diff against target: |
81 lines (+36/-7) 1 file modified
softwarecenter/ui/gtk3/app.py (+36/-7) |
To merge this branch: | bzr merge lp://qastaging/~mvo/software-center/lp914393 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Gary Lasker (community) | Approve | ||
Review via email: mp+101884@code.qastaging.launchpad.net |
Description of the change
Based on lp:~mvo/software-center/lp772549 this branch adds a explicit sys.exit() to ensure all
gtk event processing stops. This works around the issue in bug #914393 where it appears that:
softwarecenter.
self.icon.
even though the icon is already destroyed in the gtk/C world but not in the pygi/python world.
To post a comment you must log in.
Please note that the root of this crash is hopefully addressed with my patch for gtk+ in /bugs.launchpad .net/ubuntu/ +source/ apport/ +bug/938090/ comments/ 11
https:/
but even with that we should still merge this branch to avoid getting hit by similar problems
(and because AFAICT there is no reason to exit right away).