Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 194794 - net-dialup/freeradius-1.1.7 - check-radiusd-config fix
Summary: net-dialup/freeradius-1.1.7 - check-radiusd-config fix
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Alin Năstac (RETIRED)
URL: http://lists.cistron.nl/pipermail/fre...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-10-05 11:36 UTC by Matt Courtney
Modified: 2007-10-08 08:27 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Updated version of the script that gets installed to /usr/sbin/check-radiusd-config as described in the bug (freeradius-1.1.7-check-radiusd-config-gentoo.patch,1.78 KB, patch)
2007-10-05 11:37 UTC, Matt Courtney
Details | Diff
New ebuild for net-dialup/freeradius-1.1.7 that includes the patch mentioned in this bug (freeradius-1.1.7-r1.ebuild,3.91 KB, text/plain)
2007-10-05 11:38 UTC, Matt Courtney
Details
diff showing the changes I made to the new ebuild (attached) from the existing one (freeradius-1.1.7.ebuild.patch,356 bytes, patch)
2007-10-05 11:39 UTC, Matt Courtney
Details | Diff
Updated init script (radius.init,1.86 KB, text/plain)
2007-10-05 22:09 UTC, Matt Courtney
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Matt Courtney 2007-10-05 11:36:02 UTC
Hi,

The current /usr/sbin/check-radiusd-config script reports errors if radiusd is running (even if there are no errors in the config file).

I've taken a suggested patch from the freeradius-users mailing list so the script works regardless of whether or not radiusd is running.

I've also made changes to the init script so it will check the config before reloading it.

This is the first ebuild I've submitted so I apologise in advance for any errors I've made. I hope you find it useful.

Greetz,

Matt
Comment 1 Matt Courtney 2007-10-05 11:37:56 UTC
Created attachment 132643 [details, diff]
Updated version of the script that gets installed to /usr/sbin/check-radiusd-config as described in the bug
Comment 2 Matt Courtney 2007-10-05 11:38:50 UTC
Created attachment 132645 [details]
New ebuild for net-dialup/freeradius-1.1.7 that includes the patch mentioned in this bug
Comment 3 Matt Courtney 2007-10-05 11:39:41 UTC
Created attachment 132646 [details, diff]
diff showing the changes I made to the new ebuild (attached) from the existing one
Comment 4 Matt Courtney 2007-10-05 22:09:47 UTC
Created attachment 132685 [details]
Updated init script

Sorry forgot to attach the updated init script last night
Comment 5 Alin Năstac (RETIRED) gentoo-dev 2007-10-08 08:27:04 UTC
I agree with Dave Plonka's comments; this is a hack of a hack and therefore I decided not to apply this patch. If upstream decides to include this into the future version of freeradius, it will be preserved as such in the correspondent Gentoo package, but not before upstream release that version.

FWIW, check-radiusd-config work for our purposes, namely check the configuration sanity before starting the daemon. Furthermore, if you run check-radiusd-config while radiusd daemon is runnig, user gets an informative message about this.

Closed as WONTFIX.