All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gerd Hoffmann <kraxel@redhat.com>
To: qemu-devel@nongnu.org
Cc: berrange@redhat.com, Peter Maydell <peter.maydell@linaro.org>,
	Gerd Hoffmann <kraxel@redhat.com>
Subject: [Qemu-devel] [RfC PATCH] gtk: switch to gtk3 by default, depricate gtk2.
Date: Thu,  7 Jul 2016 16:27:40 +0200	[thread overview]
Message-ID: <1467901660-9054-1-git-send-email-kraxel@redhat.com> (raw)

This patch drops gtk version detection.  gtk3 is used unless you
explicitly ask for gtk2.  Additionally configure prints an error
message in case it finds gtk2 but not gtk3.

Old behavior:
  only gtk3 installed     -- use gtk3
  both gtk2+3 installed   -- use gtk2
  only gtk2 installed     -- use gtk2
  no gtk installed at all -- build without gtk ui

New behavior:
  only gtk3 installed     -- use gtk3
  both gtk2+3 installed   -- use gtk3
  only gtk2 installed     -- error out
  no gtk installed at all -- build without gtk ui

It is still possible to build with gtk2, but you have to explicitly
ask for it using --with-gtkabi=2.0.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 configure | 22 ++++++++++++++--------
 1 file changed, 14 insertions(+), 8 deletions(-)

diff --git a/configure b/configure
index e9090a0..af49eed 100755
--- a/configure
+++ b/configure
@@ -2154,14 +2154,20 @@ 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
-        gtkabi=3.0
-    else
-        gtkabi=2.0
+    gtkabi=3.0
+    if !$pkg_config --exists "gtk+-3.0 >= 3.0.0" &&
+       $pkg_config --exists "gtk+-2.0 >= 2.18.0"; then
+        echo ""
+        echo "ERROR: gtk3 not found, gtk2 is present though."
+        echo ""
+        echo "We strongly recomment to install the gtk3 devel packages and"
+        echo "build qemu with gtk3.  gtk2 support is depricated and will be"
+        echo "dropped (removal scheduled for 2018)."
+        echo ""
+        echo "If it is really your intention to build qemu with gtk2 re-run"
+        echo "configure with --with-gtkabi=2.0."
+        echo ""
+        exit 1
     fi
 fi
 
-- 
1.8.3.1

             reply	other threads:[~2016-07-07 14:27 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-07 14:27 Gerd Hoffmann [this message]
2016-07-07 14:32 ` [Qemu-devel] [RfC PATCH] gtk: switch to gtk3 by default, depricate gtk2 Daniel P. Berrange
2016-07-07 14:52 ` Peter Maydell
2016-07-07 15:04   ` Daniel P. Berrange
2016-07-07 15:42     ` Gerd Hoffmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1467901660-9054-1-git-send-email-kraxel@redhat.com \
    --to=kraxel@redhat.com \
    --cc=berrange@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.