platform/filesystem_notifications/monitor/darwin/fsevents_daemon.py imports reactor, installing default instead of qt4reactor

Bug #1029636 reported by Mike McCracken
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu One Client
Status tracked in Trunk
Stable-4-0
Fix Released
Undecided
Unassigned
Trunk
Fix Released
Critical
Mike McCracken

Bug Description

On windows and darwin, the qt4reactor has to be installed first for ubuntuone-control-panel-qt to work. That code imports platform for credentials, but doesn't need filesystem_notifications.

The issue is that platform/__init__ imports monitor, installing the default reactor, before ubuntuone-control-panel-qt installs the qt4reactor. Since control-panel doesn't need filesystem_notifications, the solution is to remove the 'convenience' import of monitor into platform.

Related branches

Mike McCracken (mikemc)
Changed in ubuntuone-client:
importance: Undecided → Critical
assignee: nobody → Mike McCracken (mikemc)
status: New → In Progress
Mike McCracken (mikemc)
description: updated
tags: added: desktop+ u1-darwin u1-windows
Changed in ubuntuone-client:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.