Merge lp:~yolanda.robla/ceilometer/grizzly into lp:~openstack-ubuntu-testing/ceilometer/grizzly

Proposed by Yolanda Robla
Status: Merged
Approved by: Yolanda Robla
Approved revision: no longer in the source branch.
Merged at revision: 20
Proposed branch: lp:~yolanda.robla/ceilometer/grizzly
Merge into: lp:~openstack-ubuntu-testing/ceilometer/grizzly
Diff against target: 63 lines (+2/-39)
3 files modified
debian/changelog (+2/-1)
debian/patches/exclude-tests-when-installing.patch (+0/-37)
debian/patches/series (+0/-1)
To merge this branch: bzr merge lp:~yolanda.robla/ceilometer/grizzly
Reviewer Review Type Date Requested Status
James Page Approve
Review via email: mp+152686@code.launchpad.net

Description of the change

Removed not-needed patches

To post a comment you must log in.
Revision history for this message
James Page (james-page) wrote :

Minor nitpick - the changelog does not actually reflect the name of the file being removed.

+ * debian/patches/exclude-tests-when-isntall.patch: removed, not needed

+ * debian/patches/exclude-tests-when-installing.patch: removed, not needed

Please merge when that is fixed.

review: Approve
lp:~yolanda.robla/ceilometer/grizzly updated
20. By Yolanda Robla

debian/patches/exclude-tests-when-installing.patch: removed, not needed

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2013-03-04 14:20:41 +0000
3+++ debian/changelog 2013-03-11 15:43:21 +0000
4@@ -31,8 +31,9 @@
5 * d/control: refreshed dependencies
6 * d/files: copying pipeline.yaml
7 * d/control: fixed typo (LP: #1140309)
8+ * debian/patches/exclude-tests-when-installing.patch: removed, not needed
9
10- -- Yolanda Robla <yolanda.robla@canonical.com> Mon, 04 Mar 2013 10:46:00 +0100
11+ -- Yolanda Robla <yolanda.robla@canonical.com> Mon, 11 Mar 2013 14:23:00 +0100
12
13 ceilometer (0.1-0ubuntu1) quantal; urgency=low
14
15
16=== removed directory 'debian/patches'
17=== removed file 'debian/patches/exclude-tests-when-installing.patch'
18--- debian/patches/exclude-tests-when-installing.patch 2013-03-01 18:25:18 +0000
19+++ debian/patches/exclude-tests-when-installing.patch 1970-01-01 00:00:00 +0000
20@@ -1,37 +0,0 @@
21-From e10fd2c0961bca3e23c743d1403fd535d2328596 Mon Sep 17 00:00:00 2001
22-From: Chuck Short <chuck.short@canonical.com>
23-Date: Fri, 1 Mar 2013 10:34:39 -0600
24-Subject: [PATCH] Exclude tests when installing ceilometer
25-
26-Dont include the tests directory when installing
27-ceilometer as they might conflict with other
28-poorly installed packages on the users systems.
29-
30-Fixes LP: #1102110
31-
32-Signed-off-by: Chuck Short <chuck.short@canonical.com>
33----
34- setup.py | 2 +-
35- tests/__init__.py | 0
36- 2 files changed, 1 insertion(+), 1 deletion(-)
37- delete mode 100644 tests/__init__.py
38-
39-diff --git a/setup.py b/setup.py
40-index 9fe04fb..9135f40 100755
41---- a/setup.py
42-+++ b/setup.py
43-@@ -63,7 +63,7 @@ setuptools.setup(
44- 'Topic :: System :: Monitoring',
45- ],
46-
47-- packages=setuptools.find_packages(exclude=['bin']),
48-+ packages=setuptools.find_packages(exclude=['tests', 'bin']),
49- cmdclass=common_setup.get_cmdclass(),
50- package_data={
51- "ceilometer":
52-diff --git a/tests/__init__.py b/tests/__init__.py
53-deleted file mode 100644
54-index e69de29..0000000
55---
56-1.8.1.2
57-
58
59=== removed file 'debian/patches/series'
60--- debian/patches/series 2013-03-01 18:25:18 +0000
61+++ debian/patches/series 1970-01-01 00:00:00 +0000
62@@ -1,1 +0,0 @@
63-exclude-tests-when-installing.patch

Subscribers

People subscribed via source and target branches