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

Proposed by Mike McCracken
Status: Merged
Approved by: Brian Curtin
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
dobey (community) Approve
Diego Sarmentero (community) Approve
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.
Revision history for this message
Diego Sarmentero (diegosarmentero) wrote :

+1

review: Approve
Revision history for this message
dobey (dobey) :
review: Approve
Revision history for this message
Brian Curtin (brian.curtin) :
review: Approve

Preview Diff

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

Subscribers

People subscribed via source and target branches