Hello Henning,

On 06/26/08 16:09, Henning Westerholt wrote:
> Revision: 4419
>           http://openser.svn.sourceforge.net/openser/?rev=4419&view=rev
> Author:   henningw
> Date:     2008-06-26 06:09:14 -0700 (Thu, 26 Jun 2008)
>
> Log Message:
> -----------
> - disable radius support in the default config for the acc module, as
>   certain debian distributions (e.g. ubuntu) compile with enabled radius,
>   this leads later to errors because the default config file path is
>   '/usr/local/etc..'. If this is fixed and the existing libradius-ng
>   config file is used, further (dictionary) errors show up
> - deactivating this support greatly improve the installation experience
>   

I think the patch is partial. On the other distributions, where the 
radius acc is not compiled, the default config will throw errors, as the 
parameter is not found.

Two options can be considered:
- move radius_config out of the rad_acc define
- set radius_config parameter to null by default, forcing in this way 
the set of it in config

Cheers,
Daniel

> Modified Paths:
> --------------
>     trunk/etc/openser.cfg
>
>
> This was sent by the SourceForge.net collaborative development platform, the 
> world's largest Open Source development site.
>
> _______________________________________________
> Devel mailing list
> Devel@lists.openser.org
> http://lists.openser.org/cgi-bin/mailman/listinfo/devel
>
>   

-- 
Daniel-Constantin Mierla
http://www.asipto.com


_______________________________________________
Devel mailing list
Devel@lists.openser.org
http://lists.openser.org/cgi-bin/mailman/listinfo/devel

Reply via email to