https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=822338
kamailio-radius-modules depends on libfreeradius-client libfreeradius-client has been deprecated in favor of radcli[1] You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/595
-- Commit Summary --
* pkg/kamailio/deb: use libradcli-dev for radius
-- File Changes --
M pkg/kamailio/deb/debian/backports/precise (2) M pkg/kamailio/deb/debian/backports/squeeze (2) M pkg/kamailio/deb/debian/backports/wheezy (2) M pkg/kamailio/deb/debian/control (2) M pkg/kamailio/deb/debian/kamailio.examples (1) M pkg/kamailio/deb/jessie/control (2) M pkg/kamailio/deb/jessie/kamailio.examples (1) M pkg/kamailio/deb/precise/control (2) M pkg/kamailio/deb/precise/kamailio.examples (1) M pkg/kamailio/deb/sid/control (2) M pkg/kamailio/deb/sid/kamailio.examples (1) M pkg/kamailio/deb/squeeze/control (2) M pkg/kamailio/deb/squeeze/kamailio.examples (1) M pkg/kamailio/deb/stretch/control (2) M pkg/kamailio/deb/stretch/kamailio.examples (1) M pkg/kamailio/deb/trusty/control (2) M pkg/kamailio/deb/trusty/kamailio.examples (1) M pkg/kamailio/deb/wheezy/control (2) M pkg/kamailio/deb/wheezy/kamailio.examples (1)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/595.patch https://github.com/kamailio/kamailio/pull/595.diff
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/595
So no changes required to radius modules -- only to pkg specs, right?
The PR can be merged. Thanks.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/595#issuecomment-216474139
Still under testing
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/595#issuecomment-216474367
So no changes required to radius modules -- only to pkg specs, right?
Sadly, no. The Debian package doesn't provide the compatible include files freeradius-client.h nor radiusclient-ng.h, so we have to options:
1. remove radiusclient-ng support and use radcli instead 2. use another define to select the proper radius library ( RADCLI ?)
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/595#issuecomment-217393761
A define is better because the radiusclient-ng is the only one available in some (old) OSes.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/595#issuecomment-217396346
Reporting in... For Fedora 23, the following can be accomplished with just a spec file change: ``` %package radius Summary: Radius AAA API for Kamailio Group: Applications/Internet Requires: kamailio = %{version}-%{release} #Requires: radiusclient-ng #BuildRequires: radiusclient-ng-devel Requires: radcli BuildRequires: radcli-compat-devel ```
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/595#issuecomment-217677469
A define is better because the radiusclient-ng is the only one available in some (old) OSes.
defined RADCLI
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/595#issuecomment-218070163
@miconda can this changes be included in 4.4.1?
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/595#issuecomment-218070562
Yes, please.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/595#issuecomment-218083625
Merged #595.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/595#event-655392208
-
Anthony Messina -
Daniel-Constantin Mierla
-
Victor Seva