Merge lp:~nataliabidart/ubuntuone-control-panel/import-qt4reactor into lp:ubuntuone-control-panel

Proposed by Natalia Bidart
Status: Merged
Approved by: Natalia Bidart
Approved revision: 245
Merged at revision: 238
Proposed branch: lp:~nataliabidart/ubuntuone-control-panel/import-qt4reactor
Merge into: lp:ubuntuone-control-panel
Diff against target: 194 lines (+58/-75)
3 files modified
ubuntuone/controlpanel/gui/qt/main/__init__.py (+41/-7)
ubuntuone/controlpanel/gui/qt/main/linux.py (+7/-34)
ubuntuone/controlpanel/gui/qt/main/windows.py (+10/-34)
To merge this branch: bzr merge lp:~nataliabidart/ubuntuone-control-panel/import-qt4reactor
Reviewer Review Type Date Requested Status
Alejandro J. Cura (community) Approve
Review via email: mp+81980@code.launchpad.net

Commit message

- Import qt4reactor properly (LP: #888722).

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

Looks good.
Tested IRL on windows

review: Approve
Revision history for this message
Ubuntu One Auto Pilot (otto-pilot) wrote :
Download full text (1.0 MiB)

The attempt to merge lp:~nataliabidart/ubuntuone-control-panel/import-qt4reactor into lp:ubuntuone-control-panel failed. Below is the output from the failed tests.

running build
Compiled data/qt/loadingoverlay.ui into ubuntuone/controlpanel/gui/qt/ui/loadingoverlay_ui.py
Compiled data/qt/preferences.ui into ubuntuone/controlpanel/gui/qt/ui/preferences_ui.py
Compiled data/qt/signin.ui into ubuntuone/controlpanel/gui/qt/ui/signin_ui.py
compiled data/qt/images.qrc into ubuntuone/controlpanel/gui/qt/ui/images_rc.py
Compiled data/qt/folders.ui into ubuntuone/controlpanel/gui/qt/ui/folders_ui.py
Compiled data/qt/devices.ui into ubuntuone/controlpanel/gui/qt/ui/devices_ui.py
Compiled data/qt/filesyncstatus.ui into ubuntuone/controlpanel/gui/qt/ui/filesyncstatus_ui.py
Compiled data/qt/device.ui into ubuntuone/controlpanel/gui/qt/ui/device_ui.py
Compiled data/qt/mainwindow.ui into ubuntuone/controlpanel/gui/qt/ui/mainwindow_ui.py
Compiled data/qt/controlpanel.ui into ubuntuone/controlpanel/gui/qt/ui/controlpanel_ui.py
Compiled data/qt/account.ui into ubuntuone/controlpanel/gui/qt/ui/account_ui.py
running build_py
creating build
creating build/lib.linux-i686-2.7
creating build/lib.linux-i686-2.7/ubuntuone
copying ubuntuone/__init__.py -> build/lib.linux-i686-2.7/ubuntuone
creating build/lib.linux-i686-2.7/ubuntuone/controlpanel
copying ubuntuone/controlpanel/login_client.py -> build/lib.linux-i686-2.7/ubuntuone/controlpanel
copying ubuntuone/controlpanel/__init__.py -> build/lib.linux-i686-2.7/ubuntuone/controlpanel
copying ubuntuone/controlpanel/logger.py -> build/lib.linux-i686-2.7/ubuntuone/controlpanel
copying ubuntuone/controlpanel/dbus_service.py -> build/lib.linux-i686-2.7/ubuntuone/controlpanel
copying ubuntuone/controlpanel/replication_client.py -> build/lib.linux-i686-2.7/ubuntuone/controlpanel
copying ubuntuone/controlpanel/cache.py -> build/lib.linux-i686-2.7/ubuntuone/controlpanel
copying ubuntuone/controlpanel/backend.py -> build/lib.linux-i686-2.7/ubuntuone/controlpanel
copying ubuntuone/controlpanel/utils.py -> build/lib.linux-i686-2.7/ubuntuone/controlpanel
creating build/lib.linux-i686-2.7/ubuntuone/controlpanel/gui
copying ubuntuone/controlpanel/gui/__init__.py -> build/lib.linux-i686-2.7/ubuntuone/controlpanel/gui
creating build/lib.linux-i686-2.7/ubuntuone/controlpanel/gui/gtk
copying ubuntuone/controlpanel/gui/gtk/package_manager.py -> build/lib.linux-i686-2.7/ubuntuone/controlpanel/gui/gtk
copying ubuntuone/controlpanel/gui/gtk/__init__.py -> build/lib.linux-i686-2.7/ubuntuone/controlpanel/gui/gtk
copying ubuntuone/controlpanel/gui/gtk/gui.py -> build/lib.linux-i686-2.7/ubuntuone/controlpanel/gui/gtk
copying ubuntuone/controlpanel/gui/gtk/widgets.py -> build/lib.linux-i686-2.7/ubuntuone/controlpanel/gui/gtk
creating build/lib.linux-i686-2.7/ubuntuone/controlpanel/gui/qt
copying ubuntuone/controlpanel/gui/qt/device.py -> build/lib.linux-i686-2.7/ubuntuone/controlpanel/gui/qt
copying ubuntuone/controlpanel/gui/qt/filesyncstatus.py -> build/lib.linux-i686-2.7/ubuntuone/controlpanel/gui/qt
copying ubuntuone/controlpanel/gui/qt/loadingoverlay.py -> build/lib.linux-i686-2.7/ubuntuone/controlpanel/gui/qt
copying ubuntuone/controlpanel/...

245. By Natalia Bidart

Fix lint issue.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'ubuntuone/controlpanel/gui/qt/main/__init__.py'
--- ubuntuone/controlpanel/gui/qt/main/__init__.py 2011-07-11 11:19:09 +0000
+++ ubuntuone/controlpanel/gui/qt/main/__init__.py 2011-11-14 11:56:20 +0000
@@ -1,6 +1,4 @@
1# -*- coding: utf-8 -*-1# -*- coding: utf-8 -*-
2
3# Authors: Manuel de la Pena <manuel@canonical.com>
4#2#
5# Copyright 2011 Canonical Ltd.3# Copyright 2011 Canonical Ltd.
6#4#
@@ -15,16 +13,52 @@
15#13#
16# You should have received a copy of the GNU General Public License along14# You should have received a copy of the GNU General Public License along
17# with this program. If not, see <http://www.gnu.org/licenses/>.15# with this program. If not, see <http://www.gnu.org/licenses/>.
16
18"""Provide the correct reactor and ui integration."""17"""Provide the correct reactor and ui integration."""
1918
20import sys19import sys
2120
22# pylint: disable=C010321from PyQt4 import QtGui
23main = None22
2423# Module used to include the resources into this file
24# Unused import images_rc, pylint: disable=W0611
25from ubuntuone.controlpanel.gui.qt.ui import images_rc
26# pylint: enable=W0611
27
28
29# Invalid name "source", pylint: disable=C0103
25if sys.platform == 'win32':30if sys.platform == 'win32':
26 from ubuntuone.controlpanel.gui.qt.main import windows31 from ubuntuone.controlpanel.gui.qt.main import windows
27 main = windows.main32 source = windows
28else:33else:
29 from ubuntuone.controlpanel.gui.qt.main import linux34 from ubuntuone.controlpanel.gui.qt.main import linux
30 main = linux.main35 source = linux
36# pylint: enable=C0103
37
38
39def main(switch_to='', alert=False, minimized=False, with_icon=False):
40 """Start the Qt reactor and open the main window."""
41 # The following cannot be imported outside this function
42 # because u1trial already provides a reactor.
43
44 # The main loop MUST be initialized before importing the reactor
45 app = QtGui.QApplication(sys.argv)
46 source.main(app)
47
48 # Reimport 'qt4reactor', 'reactor', 'start', pylint: disable=W0404, F0401
49 import qt4reactor
50 qt4reactor.install()
51
52 from twisted.internet import reactor
53 from ubuntuone.controlpanel.gui.qt.gui import start
54 # pylint: enable=W0404, F0401
55
56 # Module 'reactor' has no 'run'/'stop' member, pylint: disable=E1101
57
58 # Unused variable 'window', 'icon', pylint: disable=W0612
59 icon, window = start(reactor.stop,
60 minimized=minimized, with_icon=with_icon)
61 # pylint: enable=W0612
62
63 reactor.run()
64 # pylint: enable=E1101
3165
=== modified file 'ubuntuone/controlpanel/gui/qt/main/linux.py'
--- ubuntuone/controlpanel/gui/qt/main/linux.py 2011-09-15 16:34:02 +0000
+++ ubuntuone/controlpanel/gui/qt/main/linux.py 2011-11-14 11:56:20 +0000
@@ -1,7 +1,4 @@
1# -*- coding: utf-8 -*-1# -*- coding: utf-8 -*-
2
3# Authors: Manuel de la Pena <manuel@canonical.com>
4# Authors: Alejandro J. Cura <alecu@canonical.com>
5#2#
6# Copyright 2011 Canonical Ltd.3# Copyright 2011 Canonical Ltd.
7#4#
@@ -16,42 +13,18 @@
16#13#
17# You should have received a copy of the GNU General Public License along14# You should have received a copy of the GNU General Public License along
18# with this program. If not, see <http://www.gnu.org/licenses/>.15# with this program. If not, see <http://www.gnu.org/licenses/>.
16
19"""Main method to be used on linux."""17"""Main method to be used on linux."""
2018
21import sys
22
23from dbus.mainloop.qt import DBusQtMainLoop19from dbus.mainloop.qt import DBusQtMainLoop
24from PyQt4 import Qt, QtCore20from PyQt4 import QtCore
2521
26# Module used to include the resources into this file22
27# pylint: disable=W061123def main(app):
28from ubuntuone.controlpanel.gui.qt.ui import images_rc24 """Apply style sheet."""
29# pylint: enable=W0611
30
31
32def main(switch_to='', alert=False, minimized=False, with_icon=False):
33 """Start the Qt reactor and open the main window."""
34
35 # The DBus main loop MUST be initialized before importing the reactor25 # The DBus main loop MUST be initialized before importing the reactor
36 DBusQtMainLoop(set_as_default=True)26 DBusQtMainLoop(set_as_default=True)
3727
38 # The app variable should be in scope of the reactor.run()28 # Apply Style Sheet -- The linux version may be different
39 # pylint: disable=W0612
40 app = Qt.QApplication(sys.argv)
41
42 # Apply Style Sheet -- ONLY FOR DEBUGGING PURPOSES
43 qss = QtCore.QResource(":/ubuntuone.qss")29 qss = QtCore.QResource(":/ubuntuone.qss")
44 app.setStyleSheet(qss.data())30 app.setStyleSheet(qss.data())
45
46 # The following cannot be imported outside this function
47 # because u1trial already provides a reactor.
48 # pylint: disable=W0404, F0401
49 from qtreactor import qt4reactor
50 qt4reactor.install()
51 from twisted.internet import reactor
52 from ubuntuone.controlpanel.gui.qt.gui import start
53
54 # pylint: disable=E1101
55 icon, window = start(reactor.stop,
56 minimized=minimized, with_icon=with_icon)
57 reactor.run()
5831
=== modified file 'ubuntuone/controlpanel/gui/qt/main/windows.py'
--- ubuntuone/controlpanel/gui/qt/main/windows.py 2011-09-15 14:42:24 +0000
+++ ubuntuone/controlpanel/gui/qt/main/windows.py 2011-11-14 11:56:20 +0000
@@ -1,6 +1,4 @@
1# -*- coding: utf-8 -*-1# -*- coding: utf-8 -*-
2
3# Authors: Manuel de la Pena <manuel@canonical.com>
4#2#
5# Copyright 2011 Canonical Ltd.3# Copyright 2011 Canonical Ltd.
6#4#
@@ -15,42 +13,20 @@
15#13#
16# You should have received a copy of the GNU General Public License along14# You should have received a copy of the GNU General Public License along
17# with this program. If not, see <http://www.gnu.org/licenses/>.15# with this program. If not, see <http://www.gnu.org/licenses/>.
18"""Reactor installation for windows."""16
1917"""Main method to be used on windows."""
20import sys18
2119from PyQt4 import QtGui, QtCore
22from PyQt4 import Qt, QtGui, QtCore20
2321
24# Module used to include the resources into this file22def main(app):
25# pylint: disable=W061123 """Apply style sheet and fonts."""
26from ubuntuone.controlpanel.gui.qt.ui import images_rc
27# pylint: enable=W0611
28
29
30def main(switch_to='', alert=False, minimized=False, with_icon=False):
31 """Start the Qt reactor and open the main window."""
32
33 # The following cannot be imported outside this function
34 # because u1trial already provides a reactor.
35 # pylint: disable=F0401, W0404
36 import qtreactor.qt4reactor
37 qtreactor.qt4reactor.install()
38 from twisted.internet import reactor
39 from ubuntuone.controlpanel.gui.qt.gui import start
40
41 # The main loop MUST be initialized before importing the reactor
42 # pylint: disable=W0612
43 app = Qt.QApplication(sys.argv)
44 # Apply font to the entire application24 # Apply font to the entire application
45 QtGui.QFontDatabase.addApplicationFont(':/Ubuntu-R.ttf')25 QtGui.QFontDatabase.addApplicationFont(':/Ubuntu-R.ttf')
46 QtGui.QFontDatabase.addApplicationFont(':/Ubuntu-B.ttf')26 QtGui.QFontDatabase.addApplicationFont(':/Ubuntu-B.ttf')
47 # Apply Style Sheet27
28 # Apply Style Sheet -- The windows version may be different
48 qss_file = QtCore.QFile(":/ubuntuone.qss")29 qss_file = QtCore.QFile(":/ubuntuone.qss")
49 qss_file.open(QtCore.QFile.ReadOnly)30 qss_file.open(QtCore.QFile.ReadOnly)
50 stylesheet = QtCore.QLatin1String(qss_file.readAll())31 stylesheet = QtCore.QLatin1String(qss_file.readAll())
51 app.setStyleSheet(stylesheet)32 app.setStyleSheet(stylesheet)
52
53 icon, window = start(reactor.stop,
54 minimized=minimized, with_icon=with_icon)
55 # pylint: disable=E1101
56 reactor.run()

Subscribers

People subscribed via source and target branches