Re: [libvirt] [PATCH] 6/8 Convert qemud to the logging infrastructure

[Daniel Veillard <veillard redhat com>]

On Wed, Dec 17, 2008 at 04:28:04PM +0000, Daniel P. Berrange wrote:
> On Wed, Dec 17, 2008 at 04:26:25PM +0100, Daniel Veillard wrote:
> >   This is a bit more massive because qemud already had a limited
> > logging primitive, so we remove the old one, put the new one, initialize
> > the logging at the beginning of remoteReadConfigFile (maybe that should
> > be renamed, I think it's launched in all cases).
> >   The painful part is to convert the whole set of existing log calls
> > to the new macro,
> 
> ACK.

  I just noticed I forgot to take into account --verbose,
so in qemudSetLogging I will also add

    if ((verbose) && (log_level >= VIR_LOG_WARN))
            log_level = VIR_LOG_INFO;

to allow the command line option to override the default level.

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel veillard com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/