sql.conf vs. modules/sql

[Alan DeKok]

Stefan Winter wrote:
> when setting up a brand-new server, I fell over one caveat (and did so
> more than once, actually): almost all modules are placed in modules/*
> and get get auto-loaded when they are needed by the actual configuration.
> 
> Except eap.conf and sql.conf - these need explicit $INCLUDEs in
> radiusd.conf.

  Yes...

> I wonder why - couldn't sql.conf just become modules/sql? It's commented
> out anyway, so wouldn't get loaded with the default config. But it would
> enable auto-loading when uncommenting "sql" - making thinking of
> radiusd.conf obsolete.
> 
> What do you think?

  Probably for 2.2.0.  If people think it's an OK change to make for
2.1.11, it could be done there, too.

  Alan DeKok.