[Pkg-freeradius-maintainers] Bug#900064: freeradius: Please consider to split out python2 module into a separate package

[Dimitri John Ledkov]

Source: freeradius
Version: Please consider to split out python2 module into a separate package
Severity: wishlist

Dear Maintainer,

In anticipation of introducing python3 module, and python2
depreciation, please consider to split out python freeradius module
into a separate subpackage. This is similar to how e.g. -postgres /
-mysql / etc modules live in separate submodules. This will enable one
to install freeradius without having python2 installed on the system,
which may be desired by some sysadmins going forward.

See the attached proposed patch.

Note, depending on the version numbers actually uploaded, Breaks &
Replaces version numbers may require adjustments.

Regards,

Dimitri.
-------------- next part --------------
diff -Nru freeradius-3.0.16+dfsg/debian/changelog freeradius-3.0.16+dfsg/debian/changelog
--- freeradius-3.0.16+dfsg/debian/changelog	2018-03-20 06:52:46.000000000 +0000
+++ freeradius-3.0.16+dfsg/debian/changelog	2018-05-23 18:34:57.000000000 +0100
@@ -1,3 +1,9 @@
+freeradius (3.0.16+dfsg-4) unstable; urgency=medium
+
+  * Split out python2 freeradius module into a standalone package.
+
+ -- Dimitri John Ledkov <xnox at ubuntu.com>  Wed, 23 May 2018 18:34:57 +0100
+
 freeradius (3.0.16+dfsg-3) unstable; urgency=medium
 
   * Change default /etc/freeradius permission from 2751 to 2750 (Closes: #890933)
diff -Nru freeradius-3.0.16+dfsg/debian/control freeradius-3.0.16+dfsg/debian/control
--- freeradius-3.0.16+dfsg/debian/control	2018-03-20 06:52:46.000000000 +0000
+++ freeradius-3.0.16+dfsg/debian/control	2018-05-23 17:26:23.000000000 +0100
@@ -50,7 +50,7 @@
 Depends: lsb-base (>= 3.1-23.2), ${shlibs:Depends}, ${misc:Depends}, ${dist:Depends}, freeradius-common, freeradius-config, libfreeradius3 (= ${binary:Version})
 Provides: radius-server
 Recommends: freeradius-utils
-Suggests: freeradius-ldap, freeradius-postgresql, freeradius-mysql, freeradius-krb5, snmp
+Suggests: freeradius-ldap, freeradius-postgresql, freeradius-mysql, freeradius-krb5, snmp, freeradius-python2
 Description: high-performance and highly configurable RADIUS server
  FreeRADIUS is a high-performance RADIUS server with support for:
   - Authentication by local files, SQL, Kerberos, LDAP, PAM, and more.
@@ -190,3 +190,12 @@
 Description: Yubikey module for FreeRADIUS server
  This package is required to add Yubikey functionality to the
  FreeRADIUS server.
+
+Package: freeradius-python2
+Architecture: any
+Depends: freeradius (= ${binary:Version}), ${shlibs:Depends}, ${misc:Depends}, ${dist:Depends}
+Breaks: freeradius (<< 3.0.16+dfsg-4~)
+Replaces: freeradius (<< 3.0.16+dfsg-4~)
+Description: Python 2 module for FreeRADIUS server
+ This package is required to add Python 2 functionality to the
+ FreeRADIUS server.
diff -Nru freeradius-3.0.16+dfsg/debian/freeradius.install freeradius-3.0.16+dfsg/debian/freeradius.install
--- freeradius-3.0.16+dfsg/debian/freeradius.install	2018-03-20 06:52:46.000000000 +0000
+++ freeradius-3.0.16+dfsg/debian/freeradius.install	2018-05-23 18:34:57.000000000 +0100
@@ -34,7 +34,6 @@
 usr/lib/freeradius/rlm_passwd.so
 usr/lib/freeradius/rlm_perl.so
 usr/lib/freeradius/rlm_preprocess.so
-usr/lib/freeradius/rlm_python.so
 usr/lib/freeradius/rlm_radutmp.so
 usr/lib/freeradius/rlm_realm.so
 usr/lib/freeradius/rlm_replicate.so
diff -Nru freeradius-3.0.16+dfsg/debian/freeradius-python2.install freeradius-3.0.16+dfsg/debian/freeradius-python2.install
--- freeradius-3.0.16+dfsg/debian/freeradius-python2.install	1970-01-01 01:00:00.000000000 +0100
+++ freeradius-3.0.16+dfsg/debian/freeradius-python2.install	2018-05-23 17:26:23.000000000 +0100
@@ -0,0 +1 @@
+usr/lib/freeradius/rlm_python.so
diff -Nru freeradius-3.0.16+dfsg/debian/freeradius-python2.lintian-overrides freeradius-3.0.16+dfsg/debian/freeradius-python2.lintian-overrides
--- freeradius-3.0.16+dfsg/debian/freeradius-python2.lintian-overrides	1970-01-01 01:00:00.000000000 +0100
+++ freeradius-3.0.16+dfsg/debian/freeradius-python2.lintian-overrides	2018-05-23 17:26:23.000000000 +0100
@@ -0,0 +1,3 @@
+# There's plenty in the description of this package to identify
+# what it does.
+freeradius-python2: extended-description-is-probably-too-short
diff -Nru freeradius-3.0.16+dfsg/debian/freeradius-python2.postinst freeradius-3.0.16+dfsg/debian/freeradius-python2.postinst
--- freeradius-3.0.16+dfsg/debian/freeradius-python2.postinst	1970-01-01 01:00:00.000000000 +0100
+++ freeradius-3.0.16+dfsg/debian/freeradius-python2.postinst	2018-05-23 17:26:23.000000000 +0100
@@ -0,0 +1,14 @@
+#!/bin/sh
+# vim:ts=2:sw=2:et
+
+set -e
+
+case "$1" in
+  configure)
+    invoke-rc.d freeradius force-reload
+    ;;
+esac
+
+#DEBHELPER#
+
+exit 0