Comment 2 for bug 937099

Revision history for this message
Daniel Manrique (roadmr) wrote :

I looked at this a bit, and there's a catch-all except block in user_interface.py that just "eats" the actual exception. I removed that block and this is the error I get:

2012-02-22 12:53:18,188 ERROR Exception in handler for D-Bus signal:
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/dbus/connection.py", line 230, in maybe
_handle_message
    self._handler(*args, **kwargs)
  File "/usr/lib/python2.7/dist-packages/checkbox_qt/qt_interface.py", line 65,
in onReviewTestsClicked
    self.show_url(self.report_url)
  File "/usr/lib/python2.7/dist-packages/checkbox/user_interface.py", line 181,
in show_url
    from gi.repository import Gio
  File "/usr/lib/python2.7/dist-packages/gi/__init__.py", line 23, in <module>
    from ._gi import _API, Repository
ImportError: could not import gobject (error was: ImportError('When using gi.rep
ository you must not import static modules like "gobject". Please change all occ
urrences of "import gobject" to "from gi.repository import GObject".',))

Looks like qt_interface.py does "import gobject" (i.e. doing things "the old-fashioned way"), which later conflicts with user_interface.py's "from gi.repository import Gio" ("new way"). To fix this, modify qt_interface.py so that it does:

from gi.repository import GObject

and then change references to gobject for GObject.

I'll submit a fix in a bit.