python: simplify build, some fixups
authorAlon Bar-Lev <alon.barlev@gmail.com>
Wed, 5 Apr 2017 16:47:08 +0000 (19:47 +0300)
committerJustus Winter <justus@g10code.com>
Fri, 7 Apr 2017 10:12:20 +0000 (12:12 +0200)
* lang/python/gpg/version.py.in: Rename to lang/python/version.py.in.
configure.ac: Generate version.py.in in lang/python.
* lang/python/MANIFEST.in: Include version.py explicitly.
* lang/python/gpg: Rename to 'src'.
* lang/python/Makefile.am: Do not copy source files, do not use absolute
directories, support lib64 in uninstall, clean also dist directory, use
symlink for gpg src.
* lang/python/setup.py.in: Use builddir, copy sources into builddir,
copy version.py into module.
--

Simplify build to symlink the gpg sources into builddir instead of
copying. This requires handling of version.py as generated file.

In addition apply some cleanups: Drop the absolution pathes, clean the
dist directory as well, support lib64 for sitelib at uninstall.

Signed-off-by: Alon Bar-Lev <alon.barlev@gmail.com>
31 files changed:
configure.ac
lang/python/MANIFEST.in
lang/python/Makefile.am
lang/python/setup.py.in
lang/python/src/__init__.py [moved from lang/python/gpg/__init__.py with 100% similarity]
lang/python/src/callbacks.py [moved from lang/python/gpg/callbacks.py with 100% similarity]
lang/python/src/constants/__init__.py [moved from lang/python/gpg/constants/__init__.py with 100% similarity]
lang/python/src/constants/create.py [moved from lang/python/gpg/constants/create.py with 100% similarity]
lang/python/src/constants/data/__init__.py [moved from lang/python/gpg/constants/data/__init__.py with 100% similarity]
lang/python/src/constants/data/encoding.py [moved from lang/python/gpg/constants/data/encoding.py with 100% similarity]
lang/python/src/constants/event.py [moved from lang/python/gpg/constants/event.py with 100% similarity]
lang/python/src/constants/import.py [moved from lang/python/gpg/constants/import.py with 100% similarity]
lang/python/src/constants/keylist/__init__.py [moved from lang/python/gpg/constants/keylist/__init__.py with 100% similarity]
lang/python/src/constants/keylist/mode.py [moved from lang/python/gpg/constants/keylist/mode.py with 100% similarity]
lang/python/src/constants/keysign.py [moved from lang/python/gpg/constants/keysign.py with 100% similarity]
lang/python/src/constants/md.py [moved from lang/python/gpg/constants/md.py with 100% similarity]
lang/python/src/constants/pk.py [moved from lang/python/gpg/constants/pk.py with 100% similarity]
lang/python/src/constants/protocol.py [moved from lang/python/gpg/constants/protocol.py with 100% similarity]
lang/python/src/constants/sig/__init__.py [moved from lang/python/gpg/constants/sig/__init__.py with 100% similarity]
lang/python/src/constants/sig/mode.py [moved from lang/python/gpg/constants/sig/mode.py with 100% similarity]
lang/python/src/constants/sig/notation.py [moved from lang/python/gpg/constants/sig/notation.py with 100% similarity]
lang/python/src/constants/sigsum.py [moved from lang/python/gpg/constants/sigsum.py with 100% similarity]
lang/python/src/constants/status.py [moved from lang/python/gpg/constants/status.py with 100% similarity]
lang/python/src/constants/tofu/__init__.py [moved from lang/python/gpg/constants/tofu/__init__.py with 100% similarity]
lang/python/src/constants/tofu/policy.py [moved from lang/python/gpg/constants/tofu/policy.py with 100% similarity]
lang/python/src/constants/validity.py [moved from lang/python/gpg/constants/validity.py with 100% similarity]
lang/python/src/core.py [moved from lang/python/gpg/core.py with 100% similarity]
lang/python/src/errors.py [moved from lang/python/gpg/errors.py with 100% similarity]
lang/python/src/results.py [moved from lang/python/gpg/results.py with 100% similarity]
lang/python/src/util.py [moved from lang/python/gpg/util.py with 100% similarity]
lang/python/version.py.in [moved from lang/python/gpg/version.py.in with 100% similarity]

index 7ab94e7..9974abb 100644 (file)
@@ -881,7 +881,7 @@ AC_CONFIG_FILES([lang/Makefile lang/cl/Makefile lang/cl/gpgme.asd])
 AM_COND_IF([HAVE_DOXYGEN], [AC_CONFIG_FILES([lang/qt/doc/Doxyfile])])
 AC_CONFIG_FILES(lang/qt/doc/Makefile)
 AC_CONFIG_FILES([lang/python/Makefile
-                lang/python/gpg/version.py
+                lang/python/version.py
                 lang/python/tests/Makefile])
 AC_CONFIG_FILES([lang/python/setup.py], [chmod a+x lang/python/setup.py])
 AC_OUTPUT
index ff38172..c34e84a 100644 (file)
@@ -1,4 +1,5 @@
 recursive-include examples *.py
 include gpgme.i
 include helpers.c helpers.h private.h
+include version.py
 recursive-include gpg *.py
index 42beeee..4ebd214 100644 (file)
@@ -22,28 +22,19 @@ EXTRA_DIST = \
        gpgme.i \
        helpers.c helpers.h private.h \
        examples \
-       gpg
+       src
 
 SUBDIRS = . tests
 
-COPY_FILES_GPG = \
-       $(srcdir)/gpg/callbacks.py \
-       $(srcdir)/gpg/constants \
-       $(srcdir)/gpg/core.py \
-       $(srcdir)/gpg/errors.py \
-       $(srcdir)/gpg/__init__.py \
-       $(srcdir)/gpg/results.py \
-       $(srcdir)/gpg/util.py
-
 .PHONY: prepare
 prepare: copystamp
 
 # For VPATH builds we need to copy some files because Python's
 # distutils are not VPATH-aware.
 copystamp:
-       ln -sf "$(abs_top_srcdir)/src/data.h" .
-       ln -sf "$(abs_top_builddir)/config.h" .
-       if test $(srcdir) != . ; then cp -R $(COPY_FILES_GPG) gpg ; fi
+       ln -sf "$(top_srcdir)/src/data.h" .
+       ln -sf "$(top_builddir)/config.h" .
+       ln -sf "$(srcdir)/src" gpg
        touch $@
 
 all-local: copystamp
@@ -51,7 +42,7 @@ all-local: copystamp
          PYTHON="$$1" ; shift ; \
          CFLAGS="$(CFLAGS)" \
          srcdir="$(srcdir)" \
-         abs_top_builddir="$(abs_top_builddir)" \
+         top_builddir="$(top_builddir)" \
            $$PYTHON setup.py build --verbose --build-base=python$${VERSION}-gpg ; \
        done
 
@@ -59,7 +50,7 @@ python$(PYTHON_VERSION)-gpg/dist/gpg-$(VERSION).tar.gz.asc: copystamp
        $(MKDIR_P) python$(PYTHON_VERSION)-gpg-dist
        CFLAGS="$(CFLAGS)" \
        srcdir="$(srcdir)" \
-       abs_top_builddir="$(abs_top_builddir)" \
+       top_builddir="$(top_builddir)" \
          $(PYTHON) setup.py sdist --verbose --dist-dir=python$(PYTHON_VERSION)-gpg-dist \
                --manifest=python$(PYTHON_VERSION)-gpg-dist/MANIFEST
        gpg2 --detach-sign --armor python$(PYTHON_VERSION)-gpg-dist/gpg-$(VERSION).tar.gz
@@ -75,6 +66,7 @@ upload: python$(PYTHON_VERSION)-gpg-dist/gpg-$(VERSION).tar.gz \
 CLEANFILES = copystamp \
        config.h \
        data.h \
+       gpg \
        files.txt \
        install_files.txt
 
@@ -84,22 +76,16 @@ CLEANFILES = copystamp \
 # permissions.
 clean-local:
        rm -rf -- build
-       if test $(srcdir) != . ; then \
-         find gpg -type d ! -perm -200 -exec chmod u+w {} ';' ; \
-         for FILE in $(COPY_FILES_GPG); do \
-           rm -rf -- gpg/$$(basename $$FILE) ; \
-         done \
-       fi
        for VERSION in $(PYTHON_VERSIONS); do \
-         find python$${VERSION}-gpg -type d ! -perm -200 -exec chmod u+w {} ';' ; \
-         rm -rf -- python$${VERSION}-gpg ; \
+         find python$${VERSION}-gpg* -type d ! -perm -200 -exec chmod u+w {} ';' ; \
+         rm -rf -- python$${VERSION}-gpg* ; \
        done
 
 install-exec-local:
        set -e ; set $(PYTHONS); for VERSION in $(PYTHON_VERSIONS); do \
          PYTHON="$$1" ; shift ; \
          srcdir="$(srcdir)" \
-         abs_top_builddir="$(abs_top_builddir)" \
+         top_builddir="$(top_builddir)" \
          $$PYTHON setup.py \
          build \
          --build-base=python$${VERSION}-gpg \
@@ -110,6 +96,6 @@ install-exec-local:
 
 uninstall-local:
        GV=$$(echo $(VERSION) | tr - _); for PV in $(PYTHON_VERSIONS); do \
-         rm -rf -- "$(DESTDIR)$(prefix)/lib/python$$PV/site-packages/gpg" \
-"$(DESTDIR)$(prefix)/lib/python$$PV/site-packages/gpg-$$GV-py$$PV.egg-info" ; \
+         rm -rf -- "$(DESTDIR)$(prefix)"/lib*/python$$PV/site-packages/gpg \
+"$(DESTDIR)$(prefix)"/lib*/python$$PV/site-packages/gpg-$$GV-py$$PV.egg-info ; \
        done
index 2114aaf..5d94c70 100755 (executable)
@@ -38,18 +38,17 @@ gpgme_config = ["gpgme-config"] + gpgme_config_flags
 gpgme_h = ""
 include_dirs = [os.getcwd()]
 library_dirs = []
-vpath_build = os.environ.get('srcdir', '.') != '.'
 in_tree = False
 extra_swig_opts = []
 extra_macros = dict()
 
-abs_top_builddir = os.environ.get("abs_top_builddir")
-if abs_top_builddir:
+top_builddir = os.environ.get("top_builddir")
+if top_builddir:
     # In-tree build.
     in_tree = True
-    gpgme_config = [os.path.join(abs_top_builddir, "src/gpgme-config")] + gpgme_config_flags
-    gpgme_h = os.path.join(abs_top_builddir, "src/gpgme.h")
-    library_dirs = [os.path.join(abs_top_builddir, "src/.libs")] # XXX uses libtool internals
+    gpgme_config = [os.path.join(top_builddir, "src/gpgme-config")] + gpgme_config_flags
+    gpgme_h = os.path.join(top_builddir, "src/gpgme.h")
+    library_dirs = [os.path.join(top_builddir, "src/.libs")] # XXX uses libtool internals
     extra_macros.update(
         HAVE_CONFIG_H=1,
         HAVE_DATA_H=1,
@@ -210,23 +209,21 @@ class BuildExtFirstHack(build):
         # Copy due to http://bugs.python.org/issue2624
         # Avoid creating in srcdir
         for source, target in ((in_srcdir(n), in_build_base(n))
-                               for n in ('gpgme.i', 'helpers.c')):
+                               for n in ('gpgme.i', 'helpers.c', 'private.h', 'helpers.h')):
             if not up_to_date(source, target):
                 shutil.copy2(source, target)
 
-    def run(self):
-        self._generate()
-
         # Append generated files via build_base
         if not os.path.exists(os.path.join(self.build_lib, "gpg")):
             os.makedirs(os.path.join(self.build_lib, "gpg"))
+        shutil.copy2("version.py", os.path.join(self.build_lib, "gpg"))
+
+    def run(self):
+        self._generate()
 
         swig_sources.append(os.path.join(self.build_base, 'gpgme.i'))
         swig_opts.extend(['-I' + self.build_base,
-                          '-I' + in_srcdir('.'),
                           '-outdir', os.path.join(self.build_lib, 'gpg')])
-        if vpath_build:
-            include_dirs.append(in_srcdir('.'))
         include_dirs.append(self.build_base)
 
         self.run_command('build_ext')