Merge lp:~mandel/ubuntuone-client/reactor-new-line into lp:ubuntuone-client

Proposed by Manuel de la Peña on 2012-06-28
Status: Merged
Approved by: Manuel de la Peña on 2012-06-28
Approved revision: 1271
Merged at revision: 1270
Proposed branch: lp:~mandel/ubuntuone-client/reactor-new-line
Merge into: lp:ubuntuone-client
Diff against target: 61 lines (+5/-6)
4 files modified
tests/platform/filesystem_notifications/test_filesystem_notifications.py (+2/-0)
ubuntuone/platform/__init__.py (+0/-5)
ubuntuone/platform/os_helper/windows.py (+1/-0)
ubuntuone/syncdaemon/event_queue.py (+2/-1)
To merge this branch: bzr merge lp:~mandel/ubuntuone-client/reactor-new-line
Reviewer Review Type Date Requested Status
Diego Sarmentero (community) Approve on 2012-06-28
Facundo Batista 2012-06-28 Approve on 2012-06-28
Review via email: mp+112535@code.launchpad.net

Commit Message

- Do not import filesystem_notifications in platform so that the reactor is not installed (LP: #1018779).
- Use \N{LINE SEPARATOR} on windows to replace \n so that the files are correctly written in the file system (LP: #1017916).

Description of the Change

- Do not import filesystem_notifications in platform so that the reactor is not installed (LP: #1018779).
- Use \N{LINE SEPARATOR} on windows to replace \n so that the files are correctly written in the file system (LP: #1017916).

To post a comment you must log in.
Facundo Batista (facundo) :
review: Approve
Diego Sarmentero (diegosarmentero) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/platform/filesystem_notifications/test_filesystem_notifications.py'
2--- tests/platform/filesystem_notifications/test_filesystem_notifications.py 2012-04-30 15:19:03 +0000
3+++ tests/platform/filesystem_notifications/test_filesystem_notifications.py 2012-06-28 10:39:38 +0000
4@@ -44,6 +44,8 @@
5 remove_file,
6 remove_dir,
7 rename,
8+)
9+from ubuntuone.platform.filesystem_notifications import (
10 _GeneralINotifyProcessor,
11 )
12 from ubuntuone.syncdaemon.tritcask import Tritcask
13
14=== modified file 'ubuntuone/platform/__init__.py'
15--- ubuntuone/platform/__init__.py 2012-06-21 18:58:50 +0000
16+++ ubuntuone/platform/__init__.py 2012-06-28 10:39:38 +0000
17@@ -42,7 +42,6 @@
18 platform = "linux"
19
20 from ubuntuone.platform import ipc
21-from ubuntuone.platform import filesystem_notifications
22 from ubuntuone.platform import logger
23 from ubuntuone.platform import os_helper
24
25@@ -106,10 +105,6 @@
26 setup_filesystem_logging = logger.setup_filesystem_logging
27 get_filesystem_logger = logger.get_filesystem_logger
28
29-# From File System Notifications
30-FilesystemMonitor = filesystem_notifications.FilesystemMonitor
31-_GeneralINotifyProcessor = filesystem_notifications._GeneralINotifyProcessor
32-
33 # IPC
34 ExternalInterface = ipc.ExternalInterface
35 is_already_running = ipc.is_already_running
36
37=== modified file 'ubuntuone/platform/os_helper/windows.py'
38--- ubuntuone/platform/os_helper/windows.py 2012-05-11 17:19:44 +0000
39+++ ubuntuone/platform/os_helper/windows.py 2012-06-28 10:39:38 +0000
40@@ -108,6 +108,7 @@
41 u'|': BASE_CODE % u'\N{DIVIDES}',
42 u'?': BASE_CODE % u'\N{INTERROBANG}',
43 u'*': BASE_CODE % u'\N{SEXTILE}',
44+ u'\n': BASE_CODE % u'\N{LINE SEPARATOR}'
45 }
46 # inverse map
47 LINUX_ILLEGAL_CHARS_MAP = {}
48
49=== modified file 'ubuntuone/syncdaemon/event_queue.py'
50--- ubuntuone/syncdaemon/event_queue.py 2012-04-09 20:07:05 +0000
51+++ ubuntuone/syncdaemon/event_queue.py 2012-06-28 10:39:38 +0000
52@@ -37,7 +37,8 @@
53
54 from twisted.internet import defer
55
56-from ubuntuone.platform import access, FilesystemMonitor
57+from ubuntuone.platform.os_helper import access
58+from ubuntuone.platform.filesystem_notifications import FilesystemMonitor
59
60 # these are our internal events, what is inserted into the whole system
61 EVENTS = {

Subscribers

People subscribed via source and target branches