Merge lp:~ralsina/ubuntuone-windows-installer/stop_reactor into lp:ubuntuone-windows-installer

Proposed by Roberto Alsina
Status: Merged
Approved by: Roberto Alsina
Approved revision: 4
Merged at revision: 5
Proposed branch: lp:~ralsina/ubuntuone-windows-installer/stop_reactor
Merge into: lp:ubuntuone-windows-installer
Diff against target: 14 lines (+2/-2)
1 file modified
ubuntuone_installer/gui/qt/gui.py (+2/-2)
To merge this branch: bzr merge lp:~ralsina/ubuntuone-windows-installer/stop_reactor
Reviewer Review Type Date Requested Status
Alejandro J. Cura (community) Approve
dobey (community) Approve
Review via email: mp+65391@code.launchpad.net

Commit message

Stop the reactor once the wizard is finished.

Description of the change

Stop the reactor once the wizard is finished.

To test (on Linux):

PYTHONPATH=. python bin/ubuntuone-installer-qt

Make the wizard finish in the usual way, and the application in the terminal should stop and not hang.

To post a comment you must log in.
Revision history for this message
dobey (dobey) :
review: Approve
Revision history for this message
Alejandro J. Cura (alecu) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'ubuntuone_installer/gui/qt/gui.py'
--- ubuntuone_installer/gui/qt/gui.py 2011-06-21 16:36:01 +0000
+++ ubuntuone_installer/gui/qt/gui.py 2011-06-21 19:11:37 +0000
@@ -39,8 +39,8 @@
39 # Invalid name "closeEvent"39 # Invalid name "closeEvent"
40 # pylint: disable=C010340 # pylint: disable=C0103
4141
42 def closeEvent(self, event):42 def done(self, result):
43 """The main window is being closed, call any custom callback."""43 """The main window is being closed, call any custom callback."""
44 if self.close_callback is not None:44 if self.close_callback is not None:
45 self.close_callback()45 self.close_callback()
46 event.accept()46 QtGui.QWizard.done(self, result)

Subscribers

People subscribed via source and target branches