Hi Henning, Thanks. I will add the generic logging function in the next days, so we can start adapting the modules.
Regards, Bogdan Henning Westerholt wrote: > On Monday 17 December 2007, Bogdan-Andrei Iancu wrote: > >> Hi Henning, >> >> I think this needs a bit of a re-work as what it supposed to be in >> modules ended up in core. The solution is to have a generic LM_ function >> where you can pass facility as argument. So, the ACC facility will be a >> module parameter for acc module - the same as the "log_level". See: >> http://www.openser.org/docs/modules/1.3.x/acc.html#AEN268 >> >> So, my suggestion is to revert this commit and let me add this generic >> logging function (which anyhow is needed by other modules, like xlog) >> and to add the configurable facility as acc module param. >> > > Hi Bogdan, > > yes, this really belongs more to the module code. But this patch was getting > really stale in my repository, and i do not wanted to extend the logging > interface only for this module. I was also not aware the the xlog module > needs the same interface extension. > > I revert the commit later. > > Cheers, > > Henning > > _______________________________________________ Devel mailing list [email protected] http://lists.openser.org/cgi-bin/mailman/listinfo/devel
