[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RfC PATCH] gtk: prefer version 3.x
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [RfC PATCH] gtk: prefer version 3.x |
Date: |
Thu, 7 Jul 2016 10:29:31 +0100 |
User-agent: |
Mutt/1.6.1 (2016-04-27) |
On Thu, Jul 07, 2016 at 09:34:47AM +0200, Gerd Hoffmann wrote:
> This patch flips the default from gtk2 to gtk3.
>
> Sounds like a bigger change than it actually is as configure already
> uses gtk3 in case it doesn't find gtk2. So this changes behavior
> only in case both gtk2 and gtk3 devel packages are installed.
As a point of note, GTK 3.0 was released in 2011, so its had 5 years
to get into distros now, which is more than enough for QEMU to consider
it a valid default choice over GTK 2.x
> Signed-off-by: Gerd Hoffmann <address@hidden>
> ---
> configure | 8 +++-----
> 1 file changed, 3 insertions(+), 5 deletions(-)
>
> diff --git a/configure b/configure
> index e9090a0..e029eee 100755
> --- a/configure
> +++ b/configure
> @@ -2154,11 +2154,9 @@ fi
> # GTK probe
>
> if test "$gtkabi" = ""; then
> - # The GTK ABI was not specified explicitly, so try whether 2.0 is
> available.
> - # Use 3.0 as a fallback if that is available.
> - if $pkg_config --exists "gtk+-2.0 >= 2.18.0"; then
> - gtkabi=2.0
> - elif $pkg_config --exists "gtk+-3.0 >= 3.0.0"; then
> + # The GTK ABI was not specified explicitly, so try whether 3.0 is
> available.
> + # Use 2.0 as a fallback.
> + if $pkg_config --exists "gtk+-3.0 >= 3.0.0"; then
> gtkabi=3.0
> else
> gtkabi=2.0
Reviewed-by: Daniel P. Berrange <address@hidden>
I'd further suggest that you might want to explicitly say that GTK-2.0
support is deprecated, and set a date for its removal, as its days are
clearly numbered, and it'd let us take advantage of more advanced GTK
3.0 features without piling on yet more #ifdefs.
Perhaps print a warning in configure to say GTK-2.0 will be removed in
the first release of 2018, if people use --with-gtkabi=2.0
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|