Module: ngircd.git Branch: master Commit: 1dca082fc6f0595d6bde431bf50132445340fb7f URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=1dca0...

Author: Florian Westphal fw@strlen.de Date: Sat Jan 8 15:56:14 2011 +0100

config: deprecate NoXX-Options

ngircd unfortunately uses several options using double-negation, e.g.

NoIdent = No, NoPam = No, etc.

This renames all options by dropping the "No" prefix, e.g. "NoIdent = no" becomes "Ident = yes".

The old options will continue to work, but will cause a warning message.

Also update man pages and default config.

To prevent silly 'Ident = yes' from appearing in --configtest output in the 'ident support not compiled in and Ident Option not used' case, make default value depend on feature availability. If feature is available, enable by default, otherwise disable.

We might consider moving these options to a new [Feature]

section, or something like that, because none of these options are essential.

Another possible improvement:

'Ident = yes' option in ngircd.conf causes a warning if ngircd was built without ident support.

This does not happen with e.g. zeroconf....

---

doc/sample-ngircd.conf.tmpl | 17 +++--- man/ngircd.conf.5.tmpl | 18 +++--- src/ngircd/client.c | 2 +- src/ngircd/conf.c | 124 +++++++++++++++++++++++++++++++++---------- src/ngircd/conf.h | 16 +++--- src/ngircd/conn.c | 4 +- src/ngircd/irc-login.c | 2 +- src/ngircd/rendezvous.c | 2 +- 8 files changed, 126 insertions(+), 59 deletions(-)

Diff: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commitdiff;h=1...