]> granicus.if.org Git - ngircd/commit
config: deprecate NoXX-Options
authorFlorian Westphal <fw@strlen.de>
Sat, 8 Jan 2011 14:56:14 +0000 (15:56 +0100)
committerFlorian Westphal <fw@strlen.de>
Sun, 9 Jan 2011 12:59:33 +0000 (13:59 +0100)
commit1dca082fc6f0595d6bde431bf50132445340fb7f
tree5585848ae1c258c62786332a1508a295f3f0c25f
parent4a5dfcc3ace54de033f16503065831ed62433b2d
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
man/ngircd.conf.5.tmpl
src/ngircd/client.c
src/ngircd/conf.c
src/ngircd/conf.h
src/ngircd/conn.c
src/ngircd/irc-login.c
src/ngircd/rendezvous.c