On Mon, Jan 14, 2008 at 12:21:44PM +0000, Richard W.M. Jones wrote: > Daniel Veillard wrote: > >On Wed, Jan 09, 2008 at 05:13:47AM +0000, Daniel P. Berrange wrote: > >>During development I like to have the --enable-debug flag enabled all the > >>time because its very handy at times. I don't want it spewing to stderr > >>all the time though, just because I turned on the compile option. So this > >>patch adds an env variable 'LIBVIRT_DEBUG' which controls whether it is > >>chatty or not. So with this patch you can use --enable-debug all the time, > >>and just run > >> > >> LIBVIRT_DEBUG=1 ./src/virsh .... > >> > >>to turn it on for a particular test, or likewise for the daemon > >> > >> LIBVIRT_DEBUG=1 ./qemud/libvirtd > > > > Hum, in other projects I have found that always compiling debug support > > in > >but activating it using a command line flag or a environment variable to be > >very useful, this helps people debug stuff by themselve or improve the > >quality of the reports. Shouldn't we just do that ? it's not like anthing > >in libvirt is really timing critical or size matters that much, I can't > >see a real drawback to making --enable-debug the default, if it requires > >explicit activation. > > +1 to DV's suggestion. Ok, I'll cook up a new patch to do that. Dan. -- |=- Red Hat, Engineering, Emerging Technologies, Boston. +1 978 392 2496 -=| |=- Perl modules: http://search.cpan.org/~danberr/ -=| |=- Projects: http://freshmeat.net/~danielpb/ -=| |=- GnuPG: 7D3B9505 F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 -=| -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list