Re: [PATCH spice-common] meson: Do not define -DGLIB_VERSION_MAX_ALLOWED

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

 



On Fri, Jul 06, 2018 at 05:33:45PM +0200, Christophe Fergeau wrote:
> Ok, g_abort() is only available in 2.50, and only the meson build uses
> GLIB_VERSION_MAX_ALLOWED. However, since this can be useful, I'd prefer
> that we keep GLIB_VERSION_MAX_ALLOWED, and add it to the autotools
> build. This specific issue can be fixed by using g_error() instead of
> g_abort(). I'll send patches for that.

FYI, if you can rely on $CC always being CLang or GCC, there's a clever
trick you can do to keep GLIB_VERSION_MAX_ALLOWED while still using newer
APIs, with locally implemented back-compat wrappers. Take a look at this
QEMU example with g_strv_contains:

https://git.qemu.org/gitweb.cgi?p=qemu.git;a=blob;f=include/glib-compat.h;h=fdf95a255d9c7b8a452d76b121a7839b24ac3908;hb=HEAD#l29

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




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