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.
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 <x...@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

Reply via email to