Hi, On Wed, Jan 11, 2012 at 10:02:37AM +0100, Alan DeKok wrote:
Matthew Newton wrote:
(on the assumption that I've managed to actually drive git successfully... ;-) )
The patches look OK. The only change I would suggest is to add "#ifdef HAVE_GRP_H" around the new code. That way it will build on non-unix platforms.
Makes sense - done. rlm_detail patch: https://github.com/mcnewton/freeradius-server/commit/f9401152d4885b4a345ccb4... rlm_linelog patch: https://github.com/mcnewton/freeradius-server/commit/bb2bdd32cabcc4ff4a32da2... Cheers, Matthew -- Matthew Newton, Ph.D. <mcn4@le.ac.uk> Systems Architect (UNIX and Networks), Network Services, I.T. Services, University of Leicester, Leicester LE1 7RH, United Kingdom For IT help contact helpdesk extn. 2253, <ithelp@le.ac.uk>