From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47259) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bLAML-00034g-Ca for qemu-devel@nongnu.org; Thu, 07 Jul 2016 10:33:01 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bLAMG-000335-Kb for qemu-devel@nongnu.org; Thu, 07 Jul 2016 10:32:56 -0400 Received: from mx1.redhat.com ([209.132.183.28]:52387) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bLAMG-000331-Ed for qemu-devel@nongnu.org; Thu, 07 Jul 2016 10:32:52 -0400 Date: Thu, 7 Jul 2016 15:32:49 +0100 From: "Daniel P. Berrange" Message-ID: <20160707143249.GS31666@redhat.com> Reply-To: "Daniel P. Berrange" References: <1467901660-9054-1-git-send-email-kraxel@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1467901660-9054-1-git-send-email-kraxel@redhat.com> Subject: Re: [Qemu-devel] [RfC PATCH] gtk: switch to gtk3 by default, depricate gtk2. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Gerd Hoffmann Cc: qemu-devel@nongnu.org, Peter Maydell On Thu, Jul 07, 2016 at 04:27:40PM +0200, Gerd Hoffmann wrote: > 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 > --- > 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" s/depricated/deprecated/ > + 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 Reviewed-by: Daniel P. Berrange 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 :|