[gentoo-commits] repo/gentoo:master commit in: app-crypt/gpgme/files/

[Alon Bar-Lev]

commit:     1d4c9d162b7b4bc557c1d3b5109dec9cd2653064
Author:     Alon Bar-Lev <alonbl <AT> gentoo <DOT> org>
AuthorDate: Sat Sep 24 07:50:54 2016 +0000
Commit:     Alon Bar-Lev <alonbl <AT> gentoo <DOT> org>
CommitDate: Sat Sep 24 07:51:16 2016 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=1d4c9d16

app-crypt/gpgme: simpler python patch

Package-Manager: portage-2.2.28

 .../gpgme/files/gpgme-1.7.0-build-python.patch     | 71 ++++++++--------------
 1 file changed, 25 insertions(+), 46 deletions(-)

diff --git a/app-crypt/gpgme/files/gpgme-1.7.0-build-python.patch b/app-crypt/gpgme/files/gpgme-1.7.0-build-python.patch
index 950813f..d37d297 100644
--- a/app-crypt/gpgme/files/gpgme-1.7.0-build-python.patch
+++ b/app-crypt/gpgme/files/gpgme-1.7.0-build-python.patch
@@ -1,4 +1,4 @@
-From a46cd3926d629da23b365b70b7bf24201c25d430 Mon Sep 17 00:00:00 2001
+From e5bccea346379a70fd9cb910c4ead9642227747f Mon Sep 17 00:00:00 2001
 From: Alon Bar-Lev <alon.barlev@×××××.com>
 Date: Sat, 24 Sep 2016 01:26:19 +0300
 Subject: [PATCH] build: lang: python: cleanups
@@ -9,29 +9,21 @@ Subject: [PATCH] build: lang: python: cleanups
    anything. This is handy to enable standard distutils build outside of
    gpgme build system.
 
-3. Treat data.h in similar manner as other VPATH issues, just copy it to
-   the srcdir to simplify build, remove the CFLAGS requirement of
+3. Treat data.h in similar manner as other VPATH issues, just symlink it
+   to the srcdir to simplify build, remove the CFLAGS requirement of
    distutils.
 
-4. Add environment variable (binary_builddir) for setup to use alternate
-   build directory so that for multiple targets same binaries may be
-   used without rebuilding the C part.
-
 The above does not alter the behavior of current build but supports the
 following sequence:
 
-$ mkdir common
-$ cd common
+$ mkdir build
+$ cd build
 $ ../configure --enable-languages=
 $ make
-$ cd ..
-$ mkdir python
-$ cd python
-$ ../configure --enable-languages=
 $ cd lang/python
 $ make prepare
-$ binary_builddir="../../../common" ./setup.py build
-$ binary_builddir="../../../common" ./setup.py install
+$ ./setup.py build
+$ ./setup.py install
 
 Notice the standard use of distutils which is required for package
 manager to control the python selection and build process.
@@ -39,10 +31,9 @@ manager to control the python selection and build process.
 Signed-off-by: Alon Bar-Lev <alon.barlev@×××××.com>
 ---
  configure.ac            |  2 +-
- lang/python/Makefile.am | 11 ++++++-----
+ lang/python/Makefile.am | 16 ++++++++++------
  lang/python/gpgme.i     |  2 +-
- lang/python/setup.py.in |  9 +++++----
- 4 files changed, 13 insertions(+), 11 deletions(-)
+ 3 files changed, 12 insertions(+), 8 deletions(-)
 
 diff --git a/configure.ac b/configure.ac
 index 4a29f2f..efa1f19 100644
@@ -60,17 +51,23 @@ index 4a29f2f..efa1f19 100644
  
  echo "
 diff --git a/lang/python/Makefile.am b/lang/python/Makefile.am
-index 2271ce0..fb1eebf 100644
+index 2271ce0..9866f53 100644
 --- a/lang/python/Makefile.am
 +++ b/lang/python/Makefile.am
-@@ -51,17 +51,18 @@ copystamp: $(COPY_FILES) $(COPY_FILES_PYME)
+@@ -46,24 +46,28 @@ COPY_FILES_PYME = \
+ 
+ # For VPATH builds we need to copy some files because Python's
+ # distutils are not VPATH-aware.
+-copystamp: $(COPY_FILES) $(COPY_FILES_PYME)
++copystamp: $(COPY_FILES) $(COPY_FILES_PYME) data.h
+ 	if test "$(srcdir)" != "$(builddir)" ; then \
  	  cp -R $(COPY_FILES) . ; \
  	  cp -R $(COPY_FILES_PYME) pyme ; \
  	fi
-+	cp "$(top_srcdir)/src/data.h" .
  	touch $@
  
-+prepare: copystamp
++data.h:
++	ln -s "$(top_srcdir)/src/data.h"
 +
  all-local: copystamp
  	for PYTHON in $(PYTHONS); do \
@@ -85,8 +82,13 @@ index 2271ce0..fb1eebf 100644
 +	$(PYTHON) setup.py sdist --verbose
  	gpg2 --detach-sign --armor dist/pyme3-$(VERSION).tar.gz
  
++.PHONY: prepare
++prepare: copystamp
++
  .PHONY: sdist
-@@ -72,7 +73,7 @@ upload: dist/pyme3-$(VERSION).tar.gz dist/pyme3-$(VERSION).tar.gz.asc
+ sdist: dist/pyme3-$(VERSION).tar.gz dist/pyme3-$(VERSION).tar.gz.asc
+ 
+@@ -72,7 +76,7 @@ upload: dist/pyme3-$(VERSION).tar.gz dist/pyme3-$(VERSION).tar.gz.asc
  	twine upload $^
  
  CLEANFILES = gpgme.h errors.i gpgme_wrap.c pyme/gpgme.py \
@@ -108,29 +110,6 @@ index 84addae..ac666f4 100644
  %}
  #endif
  
-diff --git a/lang/python/setup.py.in b/lang/python/setup.py.in
-index 31892c1..ce06758 100755
---- a/lang/python/setup.py.in
-+++ b/lang/python/setup.py.in
-@@ -32,13 +32,14 @@ library_dirs = []
- in_tree = False
- extra_swig_opts = []
- extra_macros = dict()
-+binary_builddir=os.environ.get('binary_builddir', '../..')
- 
--if os.path.exists("../../src/gpgme-config"):
-+if os.path.exists(os.path.join(binary_builddir, "src/gpgme-config")):
-     # In-tree build.
-     in_tree = True
--    gpgme_config = ["../../src/gpgme-config"] + gpgme_config_flags
--    gpgme_h = "../../src/gpgme.h"
--    library_dirs = ["../../src/.libs"] # XXX uses libtool internals
-+    gpgme_config = [os.path.join(binary_builddir, "src/gpgme-config")] + gpgme_config_flags
-+    gpgme_h = os.path.join(binary_builddir, "src/gpgme.h")
-+    library_dirs = [os.path.join(binary_builddir, "src/.libs")] # XXX uses libtool internals
-     extra_macros.update(
-         HAVE_DATA_H=1,
-         IN_TREE_BUILD=1,
 -- 
 2.7.3