On Wed, 2017-02-08 at 11:42 +0100, Christophe Fergeau wrote:
> On Wed, Feb 08, 2017 at 10:57:42AM +0100, Pavel Grunt wrote:
> > On Mon, 2017-02-06 at 18:15 +0100, Christophe Fergeau wrote:
> > > Using spice_info() gets in the way of tests
> >
> > I would say that is the problem of the test, it is possible to
> > check
> > the INFO level as well. imo spice_info messages are useful
>
> The test could ignore all info messages indeed. However (on f25), I
> don't see a lot of difference between DEBUG and INFO levels, neither
> are
> shown unless G_MESSAGES_DEBUG=all is specified. Are you saying
> having
> the distinction between spice_info and spice_debug in this debug log
> is
> useful? To me, usage of one or the other seemed fairly random.
>
You are right it doesn't work. imo the info messages should be always
printed - but apparently we are using spice_printerr for that :/
There is an inconsistency, for instance
spice_printerr("add main channel client");
spice_info("create display channel");
IMHO We should get rid of spice_printerr where is used for info
messages and make spice_info work properly
Pavel
> Christophe
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel