Merge lp://qastaging/~mikemc/ubuntuone-control-panel/fix-1015825-reorder-reactor into lp://qastaging/ubuntuone-control-panel

Proposed by Mike McCracken
Status: Merged
Approved by: Alejandro J. Cura
Approved revision: 340
Merged at revision: 335
Proposed branch: lp://qastaging/~mikemc/ubuntuone-control-panel/fix-1015825-reorder-reactor
Merge into: lp://qastaging/ubuntuone-control-panel
Diff against target: 98 lines (+48/-3)
3 files modified
bin/ubuntuone-control-panel-qt (+2/-2)
ubuntuone/controlpanel/gui/qt/main/__init__.py (+13/-1)
ubuntuone/controlpanel/gui/qt/main/tests/test_main.py (+33/-0)
To merge this branch: bzr merge lp://qastaging/~mikemc/ubuntuone-control-panel/fix-1015825-reorder-reactor
Reviewer Review Type Date Requested Status
Manuel de la Peña (community) Approve
Diego Sarmentero (community) Approve
Review via email: mp+112432@code.qastaging.launchpad.net

Commit message

- On Darwin, install qt4reactor after QApplication is created. (LP: #1015825)

Description of the change

- On Darwin, install qt4reactor after QApplication is created. (LP: #1015825)

To post a comment you must log in.
339. By Mike McCracken

Refactor and add tests for installing qt4reactor in main on darwin.

340. By Mike McCracken

fix pep8 complaints

Revision history for this message
Mike McCracken (mikemc) wrote :

Tests added using alecu's suggestion to make the import testable - encapsulate import & install into a patch-able function.

Patched sys.platform in these new tests so they work on all platforms.

Revision history for this message
Diego Sarmentero (diegosarmentero) wrote :

+1

review: Approve
Revision history for this message
Manuel de la Peña (mandel) wrote :

I've ran the ui and works prefectly, I'll +1.

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