Red Hat Bugzilla – Bug 378801
Please consider adding freeradius-python subpackage
Last modified: 2008-04-17 14:23:03 EDT
Please consider adding freeradius-python subpackage.
The attached patch will enable building of rpm_python plugin and creates
separate package for it.
The module is very useful when writing custom accounting and auth systems.
I hope I will be able to change it's "experimental" status soon. The module
itself is working on my systems for couple of years now.
Created attachment 255801 [details]
Patch to enable freeradius-python subpackage
I agree, python support would be quite nice, however I'm concerned about
stability, robustness, and my limited time to field bug reports on this module.
If we ship it I have to support it.
A quick search for stability issues came up with this thread:
Those comments were from January. I don't have time at the moment to research
what might have changed since then. If you would like to do some of the leg work
on that it would help, including finding whatever patches are being talked about
and making sure they've been committed to the upstream CVS. The fact it
currently works for you is not sufficient evidence it's ready to be shipped in
If you can help nail those issues down it would help a great deal. Fedora has a
strong bias toward python and it would be appropriate to ship this module, but
only if it's ready for prime time.
Heh... If you follow the links in that thread you will see that people mention
that "patched" module works for them, and links are pointing at my messages :)
That said, my version of rlm_python was submitted to upstream this year (between
1.1.5 and 1.1.6 I think). What you have right now in upstream source is no
longer leaks memory or segfaults.
You are right and when people will start using this module new bugs could be
exposed. However I don't think they'll be serious... I can deal with them.
rlm_python is marked stable in freeradius 2.0.0, so as long as you will not
explicitly mark it as unstable, it will either go into the main package or
require a subpackage.