Bug#822337: [Pkg-nagios-devel] Bug#822337: upgrade from libfreeradius-client to radcli

2016-11-14 Thread Jan Wagner 
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA512

Dear radcli maintainers,

Am 05.11.2016 um 20:42 schrieb Jan Wagner:
> Am 03.11.2016 um 00:51 schrieb Jan Wagner:
>> monitoring-plugins configure tries to detect the radius library 
>> which can't be detected, with libradcli-dev (and libradcli4) 
>> installed:
> 
>> configure: WARNING: Skipping radius plugin configure: WARNING: 
>> install radius libs to compile this plugin (see REQUIREMENTS).
> 
> After patching this issue 
> (https://github.com/waja/monitoring-plugins/commits/radcli) the 
> problem seems, it's not so compatible like it seems 
> (https://travis-ci.org/waja/monitoring-plugins/builds/173546204#L3533-
L3
>
> 
548):
> 
> check_radius.c:95:1: error: unknown type name ‘ENV’ ENV *env =
> NULL; ^ check_radius.c: In function ‘main’: 
> check_radius.c:205:2: error: too few arguments to function 
> ‘rc_send_server’ result = my_rc_send_server (, msg); ^ In
> file included from check_radius.c:40:0: 
> /usr/include/radcli/radcli.h:651:5: note: declared here int
> rc_send_server (rc_handle *rh, SEND_DATA *data, char *msg, ^ Seems
> like radcli removed some API stuff:
> 
> https://github.com/radcli/radcli/commit/9f2da1ca9dade4bb6fb318d66f80ba
dd
>
> 
61ed1830

you might have a better insight into the (incompatible) API changes in
radcli compared to freeradius-client and can assist us?

Many thanks, Jan.
- -- 
Never write mail to , you have been warned!
- -BEGIN GEEK CODE BLOCK-
Version: 3.12
GIT d-- s+: a C+++ UL P+ L+++ E--- W+++ N+++ o++ K++ w--- O M+ V- PS
PE Y++
PGP++ t-- 5 X R tv- b+ DI D+ G++ e++ h r+++ y
- --END GEEK CODE BLOCK--
-BEGIN PGP SIGNATURE-

iQIcBAEBCgAGBQJYKhlTAAoJEAxwVXtaBlE+uSEP/jBiCCKkVNo/e/R+mpO+Rfl2
2rSvIkVJbPzjKeMMGh9J1gBXayMI8Evd/02JiTPJ/c48PtlAe1KmIYNAwCcpY4gA
v26tO573dSqDaRw5P8QWmc2LcuGBsuc+jZZLNmWHvpn6zEfZOT9IgTcI6dCw1OaS
pXnGT3x/rNBKuwFlaXEkti60dbm2bhzab7ikqTV4L9uHXobU9dJ8Xpk/c6w0wIv5
L8csG05vAKY2j609zqGkUKQXknRGNnZfdj13KXmquiHPiuBeDdHaJkMBDUEc9GdR
Phxg03zHl94YzYlv23sJeGeG1M3c4YpyBtgFA3+MQ2CJx8O+XJnC9tTRoWhVDEln
lu3l5HLWagjpaoxnhoWDrq/w1nQ/Chn71W0/5mdzY4j5bAcrXp86rSCjTQAAvwbK
FEgx84a8klvIu5x2lsphcnBwA4Xb8q36R91FophIh04J+KzRoUaoC7wNbdmI3gb7
vjz8NjaQWu9UOhOmBvosl7knfRh1EnG9jHKShXjlU2pABa2oVU93rzEmTbiux9Ka
P3xSSSDlewxRQI+n/K4ntdqh4IAF4Nq1Krs3Nqvu9vQimDN8+YsXe01QX/t6JGlJ
PTJDFwFMkjs4s8UD+yUWvdRKixfCYyRJN623KHyo8IwM1OMLnS6fIw6wOAp9Qzcw
NHpeRkxNRxsOOkC64hFc
=i9rD
-END PGP SIGNATURE-

Bug#822337: upgrade from libfreeradius-client to radcli

2016-11-05 Thread Jan Wagner 
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA512

Am 03.11.2016 um 00:51 schrieb Jan Wagner:
> monitoring-plugins configure tries to detect the radius library
> which can't be detected, with libradcli-dev (and libradcli4)
> installed:
> 
> configure: WARNING: Skipping radius plugin configure: WARNING:
> install radius libs to compile this plugin (see REQUIREMENTS).

After patching this issue
(https://github.com/waja/monitoring-plugins/commits/radcli) the
problem seems, it's not so compatible like it seems
(https://travis-ci.org/waja/monitoring-plugins/builds/173546204#L3533-L3
548):

check_radius.c:95:1: error: unknown type name ‘ENV’
 ENV *env = NULL;
 ^
check_radius.c: In function ‘main’:
check_radius.c:205:2: error: too few arguments to function
‘rc_send_server’
  result = my_rc_send_server (, msg);
  ^
In file included from check_radius.c:40:0:
/usr/include/radcli/radcli.h:651:5: note: declared here
 int rc_send_server (rc_handle *rh, SEND_DATA *data, char *msg,
 ^
Seems like radcli removed some API stuff:

https://github.com/radcli/radcli/commit/9f2da1ca9dade4bb6fb318d66f80badd
61ed1830

Cheers, Jan.
-BEGIN PGP SIGNATURE-

iQIcBAEBCgAGBQJYHjY6AAoJEAxwVXtaBlE+LxgQALumm8oGIk6Jb4UQhBMf/913
0EW0L2RLn1qUsPCKEGC+Z7qPa/Qoc5wdSxX6sZ9CTpGofuei27L0AcdbDqy45aBD
xZu6yAnq7DHirs7GuCtmXUA1dajk9ZpXRwA82jV/9Hf7OWlxVqeAQMg8gagj/W8V
1tIwL+QMFX+4rwQuXVJb0fhQgurxMg/5P3/Dg0aaGePQQHv4L06i4z/aglYF2kqM
tctlEok5ju5g3a0nI4bSrQUxXe4WoUO3xR+12VyWoVBlSfuy2Z8LW0GoeFO0veWK
W0IGJCO26qpPWsRIzE+9EOxKTt4mRWPqkFR9WGi/p+z/xzxuW1mCb/yB07Sjhitn
XnlgloxpPgJcHx3Gk19qKBwu0TlzdxZpYukssNIAIwod9hCXbtnTXi39oiEO6DS6
0IaBhVI69yEwVS2RYZUeYiw9SKBJUxPLNQCyj2AJ7q00kiOWOA5OMDtM1MpwP3yQ
yst/M40jfOr90rfMzmXlky2SDVp0a0nzmfwBf3O0a5WAs5pHeBs9Vh7FW4FJXa32
PaZk7Yn57AQDxB2MuEj+WI9KAlr2OJb6yWc1oCrpaXGbChQwQIRi5d91yKUF223y
DSCXUp6Ets8KOZxFpV6jBscuP40usBe6x1TNLfYaJolOT55ettUQG9JyJProivbm
4SoRn0PNBEZHMMu+xwpB
=y7O8
-END PGP SIGNATURE-

Bug#822337: upgrade from libfreeradius-client to radcli

2016-11-02 Thread Jan Wagner 
Am 02.11.16 um 23:38 schrieb Jan Wagner:
> it's my understanding that it's a drop in for freeradius-client
> and radiusclient-ng. So simply adding 'libradcli-dev' as the first build
> dependency alternaive should do the trick?

monitoring-plugins configure tries to detect the radius library which
can't be detected, with libradcli-dev (and libradcli4) installed:

configure: WARNING: Skipping radius plugin
configure: WARNING: install radius libs to compile this plugin (see
REQUIREMENTS).

With kind regards, Jan.
-- 
Never write mail to , you have been warned!
-BEGIN GEEK CODE BLOCK-
Version: 3.12
GIT d-- s+: a C+++ UL P+ L+++ E--- W+++ N+++ o++ K++ w--- O M+ V- PS
PE Y++
PGP++ t-- 5 X R tv- b+ DI D+ G++ e++ h r+++ y
--END GEEK CODE BLOCK--



signature.asc
Description: OpenPGP digital signature

Bug#822337: [Pkg-nagios-devel] Bug#822337: upgrade from libfreeradius-client to radcli

2016-11-02 Thread Jan Wagner 
Hi Daniel,

Am 23.04.16 um 17:43 schrieb Daniel Pocock:
> Please check the preinst for radcli to see if you are satisfied with it,
> it has to copy /etc/radiusclient/* to /etc/radcli

sorry, but the migration of configurations done by users or
radiusclient/radcli maintainers is nothing we will deal in
monitoring-plugins.
With kind regards, Jan.
-- 
Never write mail to , you have been warned!
-BEGIN GEEK CODE BLOCK-
Version: 3.12
GIT d-- s+: a C+++ UL P+ L+++ E--- W+++ N+++ o++ K++ w--- O M+ V- PS
PE Y++
PGP++ t-- 5 X R tv- b+ DI D+ G++ e++ h r+++ y
--END GEEK CODE BLOCK--



signature.asc
Description: OpenPGP digital signature

Bug#822337: upgrade from libfreeradius-client to radcli

2016-11-02 Thread Jan Wagner 
Hi Sandro,

Am 02.11.16 um 23:16 schrieb Sandro Tosi:
> On Sat, 23 Apr 2016 17:43:05 +0200 Daniel Pocock  wrote:
>> > monitoring-plugins-standard depends on libfreeradius-client
>> >
>> > libfreeradius-client has been deprecated in favor of radcli[1]
>> >
>> > Please update the build dependency to use radcli. The API is fully
>> > compatible, radcli is a fork of the previous library, it is actively
>> > maintained and includes many bug fixes.
>> >
>> > radcli is available in sid and jessie-backports
>> >
>> > Please check the preinst for radcli to see if you are satisfied with it,
>> > it has to copy /etc/radiusclient/* to /etc/radcli
> do you plan to make this change or do you see any issue with it? can i
> help somehow?

not being very familiar with all those radius stuff tooking into
https:github.com/radcli/radcli#1-introduction (based originally on
freeradius-client and radiusclient-ng and is source compatible with
them) it's my understanding that it's a drop in for freeradius-client
and radiusclient-ng. So simply adding 'libradcli-dev' as the first build
dependency alternaive should do the trick?

Cheers, Jan.
-- 
Never write mail to , you have been warned!
-BEGIN GEEK CODE BLOCK-
Version: 3.12
GIT d-- s+: a C+++ UL P+ L+++ E--- W+++ N+++ o++ K++ w--- O M+ V- PS
PE Y++
PGP++ t-- 5 X R tv- b+ DI D+ G++ e++ h r+++ y
--END GEEK CODE BLOCK--



signature.asc
Description: OpenPGP digital signature

Bug#822337: upgrade from libfreeradius-client to radcli

2016-11-02 Thread Sandro Tosi 
Hello monitoring-plugin maintainers!

On Sat, 23 Apr 2016 17:43:05 +0200 Daniel Pocock  wrote:
> monitoring-plugins-standard depends on libfreeradius-client
>
> libfreeradius-client has been deprecated in favor of radcli[1]
>
> Please update the build dependency to use radcli. The API is fully
> compatible, radcli is a fork of the previous library, it is actively
> maintained and includes many bug fixes.
>
> radcli is available in sid and jessie-backports
>
> Please check the preinst for radcli to see if you are satisfied with it,
> it has to copy /etc/radiusclient/* to /etc/radcli

do you plan to make this change or do you see any issue with it? can i
help somehow?

Bug#822337: upgrade from libfreeradius-client to radcli

2016-04-23 Thread Daniel Pocock 
Package: monitoring-plugins-standard
Severity: serious
Version: 2.1.2-2

monitoring-plugins-standard depends on libfreeradius-client

libfreeradius-client has been deprecated in favor of radcli[1]

Please update the build dependency to use radcli. The API is fully
compatible, radcli is a fork of the previous library, it is actively
maintained and includes many bug fixes.

radcli is available in sid and jessie-backports

Please check the preinst for radcli to see if you are satisfied with it,
it has to copy /etc/radiusclient/* to /etc/radcli




1. https://packages.qa.debian.org/r/radcli.html