Re: [PATCH spice-gtk] util: do not overwrite G_MESSAGES_DEBUG=all

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Oct 29, 2013 at 09:30:49PM +0100, Marc-André Lureau wrote:
> glib log handler only check for a single 'all' value, mixing it with
> other domains name will disable 'all' messages.
> ---
>  gtk/spice-util.c | 2 ++
>  spice-common     | 2 +-
>  2 files changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/gtk/spice-util.c b/gtk/spice-util.c
> index 6ac6af6..823d47b 100644
> --- a/gtk/spice-util.c
> +++ b/gtk/spice-util.c
> @@ -48,6 +48,8 @@ static void spice_util_enable_debug_messages(void)
>      const gchar *doms = g_getenv("G_MESSAGES_DEBUG");
>      if (!doms) {
>          g_setenv("G_MESSAGES_DEBUG", G_LOG_DOMAIN, 1);
> +    } else if (g_str_equal(doms, "all")) {
> +	return;

It would be nice to fix it in gtk-vnc as well. The proper fix belongs in
glib imo.

ACK.

Christophe

Attachment: pgpYcBxSsv4iG.pgp
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]