[Pkg-samba-maint] Bug#909135: consider dropping python support from libtevent.a

[Helmut Grohne]

Source: tevent
Version: 0.9.36-2
Severity: wishlist
Tags: patch
User: helmutg at debian.org
Usertags: rebootstrap

I tried cross building tevent and its Build-Depends turn out to be
"difficult" to satisfy. So I looked why they exist and was very
confused.

When building tevent with --disable-python and then comparing the
resulting packages using diffoscope, one can see that indeed there is a
difference. Surprisingly, that difference is limited to the static
libtevent.a. The shared library lacks Python support.

Given that you cannot load a static library into a Python interpreter
and that static linking is discouraged in Debian, it would seem that the
use of Python support in a static library is quite limited. How about
removing it by passing --disable-python?

The resulting simplification of Build-Depends should bring us closer to
cross building tevent.

Helmut
-------------- next part --------------
diff --minimal -Nru tevent-0.9.36/debian/changelog tevent-0.9.36/debian/changelog
--- tevent-0.9.36/debian/changelog	2018-05-15 15:42:59.000000000 +0200
+++ tevent-0.9.36/debian/changelog	2018-09-18 21:10:58.000000000 +0200
@@ -1,3 +1,10 @@
+tevent (0.9.36-2.1) UNRELEASED; urgency=medium
+
+  * Non-maintainer upload.
+  * Drop python support from the static library. (Closes: #-1)
+
+ -- Helmut Grohne <helmut at subdivi.de>  Tue, 18 Sep 2018 21:10:58 +0200
+
 tevent (0.9.36-2) unstable; urgency=low
 
   * Upload to unstable
diff --minimal -Nru tevent-0.9.36/debian/control tevent-0.9.36/debian/control
--- tevent-0.9.36/debian/control	2018-03-11 20:53:12.000000000 +0100
+++ tevent-0.9.36/debian/control	2018-09-18 21:10:58.000000000 +0200
@@ -8,8 +8,6 @@
                libtalloc-dev (>= 2.1.11~),
                pkg-config,
                python,
-               python-all-dev (>= 2.6.6-3),
-               python-talloc-dev (>= 2.1.11~)
 Standards-Version: 4.1.3
 Homepage: https://tevent.samba.org/
 Vcs-Browser: https://salsa.debian.org/samba-team/tevent
diff --minimal -Nru tevent-0.9.36/debian/rules tevent-0.9.36/debian/rules
--- tevent-0.9.36/debian/rules	2018-03-11 20:41:47.000000000 +0100
+++ tevent-0.9.36/debian/rules	2018-09-18 21:10:58.000000000 +0200
@@ -19,7 +19,7 @@
 
 override_dh_auto_configure:
 	CFLAGS="$(CFLAGS)" ./configure --prefix=/usr --bundled-libraries=NONE \
-		   --disable-rpath --disable-rpath-install \
+		   --disable-rpath --disable-rpath-install --disable-python \
 		   --minimum-library-version="$(shell ./debian/autodeps.py --minimum-library-version)" \
 		   --libdir=/usr/lib/$(DEB_HOST_MULTIARCH)