[Pkg-ceph-commits] [ceph] 23/43: d/p/skip-setup.py-makefiles.patch, rules: Avoid use of virtualenv to install ceph-disk and ceph-detect-init python modules.

[Gaudenz Steinlin]

This is an automated email from the git hooks/post-receive script.

gaudenz pushed a commit to branch jewel-gaudenz
in repository ceph.

commit 2be4148d05df7f640d4b844f48bdb3936d1f8199
Author: James Page <james.page at ubuntu.com>
Date:   Wed Mar 23 13:55:45 2016 +0000

    d/p/skip-setup.py-makefiles.patch,rules: Avoid use of virtualenv to install ceph-disk and ceph-detect-init python modules.
---
 debian/ceph.install                          |  3 ++-
 debian/patches/series                        |  1 +
 debian/patches/skip-setup.py-makefiles.patch | 20 ++++++++++++++++++++
 debian/rules                                 |  6 ++++++
 4 files changed, 29 insertions(+), 1 deletion(-)

diff --git a/debian/ceph.install b/debian/ceph.install
index ce87a75..04f13a6 100644
--- a/debian/ceph.install
+++ b/debian/ceph.install
@@ -6,7 +6,6 @@ udev/95-ceph-osd.rules lib/udev/rules.d
 usr/bin/ceph-bluefs-tool
 usr/bin/ceph-clsinfo
 usr/bin/ceph-debugpack
-usr/bin/ceph-detect-init
 usr/bin/ceph-mon
 usr/bin/ceph-objectstore-tool
 usr/bin/ceph-osd
@@ -20,8 +19,10 @@ usr/lib/*/ceph/ceph_common.sh
 usr/lib/*/ceph/erasure-code/libec_*.so
 usr/lib/*/rados-classes/*.so
 usr/lib/python*/dist-packages/ceph_detect_init*
+usr/lib/python*/dist-packages/ceph_disk*
 usr/lib/python*/dist-packages/ceph_rest_api.py
 usr/sbin/ceph-create-keys
+usr/sbin/ceph-detect-init
 usr/sbin/ceph-disk
 usr/share/doc/ceph/sample.ceph.conf
 usr/share/doc/ceph/sample.fetch_config
diff --git a/debian/patches/series b/debian/patches/series
index 3ae6cad..bb610fe 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -14,3 +14,4 @@ tests-disable.patch
 ## From Ubuntu
 fix-cycles-arch.patch
 rocksdb-flags.patch
+skip-setup.py-makefiles.patch
diff --git a/debian/patches/skip-setup.py-makefiles.patch b/debian/patches/skip-setup.py-makefiles.patch
new file mode 100644
index 0000000..9e36e35
--- /dev/null
+++ b/debian/patches/skip-setup.py-makefiles.patch
@@ -0,0 +1,20 @@
+Description: Skip Makefile base install of python modules
+ ceph-disk and ceph-detect-init are python modules; the Makefile
+ based build uses tox and virtualenv, which we really don't want
+ to use in the package build process.
+ .
+ Skip this part of the build, and use d/rules to install.
+Author: James Page <james.page at ubuntu.com>
+Forwarded: not-needed
+
+--- a/src/Makefile.am
++++ b/src/Makefile.am
+@@ -32,8 +32,6 @@ endif
+ include arch/Makefile.am
+ include auth/Makefile.am
+ include brag/Makefile.am
+-include ceph-detect-init/Makefile.am
+-include ceph-disk/Makefile.am
+ include crush/Makefile.am
+ include kv/Makefile.am
+ include mon/Makefile.am
diff --git a/debian/rules b/debian/rules
index 7e9f9de..8e51150 100755
--- a/debian/rules
+++ b/debian/rules
@@ -72,6 +72,12 @@ override_dh_install:
 	# remove .la files, we don't want to install these in Debian,
 	# see Policy 10.2
 	find debian/tmp -name '*.la' -delete
+	# NOTE: manually install python modules to avoid virtualenvs
+	#       during the package build process.
+	set -e; for comp in ceph-detect-init ceph-disk; do \
+		(set -e; cd src/$$comp; python setup.py install --root=$(CURDIR)/debian/tmp \
+			--install-script=/usr/sbin --install-layout=deb); \
+	done
 	# NOTE: ensure that any versioned erasure coding test code is dropped
 	#       from the package install - package ships unversioned modules.
 	rm -f $(CURDIR)/debian/tmp/usr/lib/*/ceph/erasure-code/libec_*.so.*

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-ceph/ceph.git