Theo Van Dinter <[EMAIL PROTECTED]> writes:

> Since we've already overloaded the add_facilities() function to also
> change level w/ "info", we could change it so that one can override
> for all the levels.  That would make it easier for people to just keep
> using "-D", but it does overload the facility API.

If I supported lowering/raising more so, I'd add a new API.
 
> It seems like the breakdown ought to be something like:
> 
> dbg           messages only seen with -D, nothing major
> info          messages only seen in logs (not STDERR), nothing major
> warning       messages logged and seen via STDERR, but processing continues 
> (warn)
> error         messages logged and seen via STDERR, processing stops (die)

I can change the code to set different levels per facility.  That's not
too hard.  I'll try to check in a fix this weekend.

The other possible addition would be to add a notice() level for most
log-centric messages in the spamd and preforking code.

> FWIW: I've already had users complaining to me about the info output.
> They'll run sa-learn, see some info output saying that a config line
> can't be parsed, and think sa-learn failed.  So the new verboseness is
> going to cause issues.

Well, they are errors!  Most Unix programs would show an error.

Daniel

-- 
Daniel Quinlan
http://www.pathname.com/~quinlan/

Reply via email to