Merge lp:~mikemc/ubuntuone-client/cleanup-platform-constants into lp:ubuntuone-client

Proposed by Mike McCracken on 2012-05-01
Status: Merged
Approved by: Brian Curtin on 2012-05-03
Approved revision: 1230
Merged at revision: 1239
Proposed branch: lp:~mikemc/ubuntuone-client/cleanup-platform-constants
Merge into: lp:ubuntuone-client
Diff against target: 89 lines (+0/-69)
3 files modified
ubuntuone/platform/linux/constants.py (+0/-34)
ubuntuone/platform/windows/__init__.py (+0/-1)
ubuntuone/platform/windows/constants.py (+0/-34)
To merge this branch: bzr merge lp:~mikemc/ubuntuone-client/cleanup-platform-constants
Reviewer Review Type Date Requested Status
Brian Curtin (community) Approve on 2012-05-03
dobey (community) Approve on 2012-05-03
Diego Sarmentero (community) 2012-05-01 Approve on 2012-05-01
Review via email: mp+104206@code.launchpad.net

Commit Message

- Refactor platform/constants in ubuntuone-client ( LP: #989119 )

Description of the Change

- Refactor platform/constants in ubuntuone-client ( LP: #989119 )

* Removed redundant files platform/windows/constants.py and platform/linux/constants.py.
platform/constants.py duplicates those files.

* Removed import of platform/windows/constants.py in platform/windows/__init__.py

* Did not create a platform/constants/ subpackage as in bug description, due to discussion on IRC that this is too small for a package.

* Tested on linux.

To post a comment you must log in.
Diego Sarmentero (diegosarmentero) wrote :

+1

review: Approve
dobey (dobey) :
review: Approve
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== removed file 'ubuntuone/platform/linux/constants.py'
2--- ubuntuone/platform/linux/constants.py 2012-04-09 20:07:05 +0000
3+++ ubuntuone/platform/linux/constants.py 1970-01-01 00:00:00 +0000
4@@ -1,34 +0,0 @@
5-#
6-# Author: Alejandro J. Cura <alecu@canonical.com>
7-#
8-# Copyright 2011-2012 Canonical Ltd.
9-#
10-# This program is free software: you can redistribute it and/or modify it
11-# under the terms of the GNU General Public License version 3, as published
12-# by the Free Software Foundation.
13-#
14-# This program is distributed in the hope that it will be useful, but
15-# WITHOUT ANY WARRANTY; without even the implied warranties of
16-# MERCHANTABILITY, SATISFACTORY QUALITY, or FITNESS FOR A PARTICULAR
17-# PURPOSE. See the GNU General Public License for more details.
18-#
19-# You should have received a copy of the GNU General Public License along
20-# with this program. If not, see <http://www.gnu.org/licenses/>.
21-#
22-# In addition, as a special exception, the copyright holders give
23-# permission to link the code of portions of this program with the
24-# OpenSSL library under certain conditions as described in each
25-# individual source file, and distribute linked combinations
26-# including the two.
27-# You must obey the GNU General Public License in all respects
28-# for all of the code used other than OpenSSL. If you modify
29-# file(s) with this exception, you may extend this exception to your
30-# version of the file(s), but you are not obligated to do so. If you
31-# do not wish to do so, delete this exception statement from your
32-# version. If you delete this exception statement from all source
33-# files in the program, then also delete it here.
34-"""Constants on the linux platform."""
35-
36-# To start hashing a file, no new hash request should arrive
37-# for it within this amount of seconds:
38-HASHQUEUE_DELAY = 0.5
39
40=== modified file 'ubuntuone/platform/windows/__init__.py'
41--- ubuntuone/platform/windows/__init__.py 2012-04-27 20:39:54 +0000
42+++ ubuntuone/platform/windows/__init__.py 2012-05-01 05:52:18 +0000
43@@ -36,7 +36,6 @@
44 platform = "win32"
45
46
47-import constants
48 import event_logging
49 import filesystem_notifications
50 import launcher
51
52=== removed file 'ubuntuone/platform/windows/constants.py'
53--- ubuntuone/platform/windows/constants.py 2012-04-09 20:07:05 +0000
54+++ ubuntuone/platform/windows/constants.py 1970-01-01 00:00:00 +0000
55@@ -1,34 +0,0 @@
56-#
57-# Author: Alejandro J. Cura <alecu@canonical.com>
58-#
59-# Copyright 2012 Canonical Ltd.
60-#
61-# This program is free software: you can redistribute it and/or modify it
62-# under the terms of the GNU General Public License version 3, as published
63-# by the Free Software Foundation.
64-#
65-# This program is distributed in the hope that it will be useful, but
66-# WITHOUT ANY WARRANTY; without even the implied warranties of
67-# MERCHANTABILITY, SATISFACTORY QUALITY, or FITNESS FOR A PARTICULAR
68-# PURPOSE. See the GNU General Public License for more details.
69-#
70-# You should have received a copy of the GNU General Public License along
71-# with this program. If not, see <http://www.gnu.org/licenses/>.
72-#
73-# In addition, as a special exception, the copyright holders give
74-# permission to link the code of portions of this program with the
75-# OpenSSL library under certain conditions as described in each
76-# individual source file, and distribute linked combinations
77-# including the two.
78-# You must obey the GNU General Public License in all respects
79-# for all of the code used other than OpenSSL. If you modify
80-# file(s) with this exception, you may extend this exception to your
81-# version of the file(s), but you are not obligated to do so. If you
82-# do not wish to do so, delete this exception statement from your
83-# version. If you delete this exception statement from all source
84-# files in the program, then also delete it here.
85-"""Constants on the windows platform."""
86-
87-# To start hashing a file, no new hash request should arrive
88-# for it within this amount of seconds:
89-HASHQUEUE_DELAY = 3.0

Subscribers

People subscribed via source and target branches