[Qemu-devel] [RfC PATCH] gtk: prefer version 3.x

[Qemu-devel] [RfC PATCH] gtk: prefer version 3.x

[Gerd Hoffmann]

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.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 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
-- 
1.8.3.1

Re: [Qemu-devel] [RfC PATCH] gtk: prefer version 3.x

[Daniel P. Berrange]

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 <kraxel@redhat.com>
> ---
>  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 <berrange@redhat.com>


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 :|

Re: [Qemu-devel] [RfC PATCH] gtk: prefer version 3.x

[Peter Maydell]

On 7 July 2016 at 10:29, Daniel P. Berrange <berrange@redhat.com> wrote:
> 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

Yeah, even debian oldstable has a gtk 3.0...

thanks
-- PMM

Re: [Qemu-devel] [RfC PATCH] gtk: prefer version 3.x

[Gerd Hoffmann]

  Hi,

> 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.

Good idea.

> 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

I guess a warning is easily overlooked.  New RfC patch follows in a
moment ...

cheers,
  Gerd