From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38332) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZQGtf-0005dA-Ae for qemu-devel@nongnu.org; Fri, 14 Aug 2015 11:27:56 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZQGtb-0005hM-C9 for qemu-devel@nongnu.org; Fri, 14 Aug 2015 11:27:55 -0400 Received: from mx1.redhat.com ([209.132.183.28]:51365) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZQGtb-0005h8-6n for qemu-devel@nongnu.org; Fri, 14 Aug 2015 11:27:51 -0400 Date: Fri, 14 Aug 2015 16:27:46 +0100 From: "Daniel P. Berrange" Message-ID: <20150814152746.GR7776@redhat.com> References: <1439565052-3457-1-git-send-email-peter.maydell@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1439565052-3457-1-git-send-email-peter.maydell@linaro.org> Subject: Re: [Qemu-devel] [PATCH] configure: Don't permit SDL or GTK on OSX Reply-To: "Daniel P. Berrange" List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: Programmingkid , Andreas =?utf-8?Q?F=C3=A4rber?= , qemu-devel@nongnu.org, patches@linaro.org On Fri, Aug 14, 2015 at 04:10:52PM +0100, Peter Maydell wrote: > The cocoa GUI frontend assumes it is the only GUI (it redefines > main() so it always gets control before the rest of QEMU), so > it does not play well with other UIs like SDL or GTK. (Mostly > people building QEMU on OSX don't have the necessary dependencies > available for configure to build those other front ends, so > mostly this problem goes unnoticed.) > > Make configure automatically disable the SDL and GTK front ends > if the cocoa front end is enabled. (We were sort of attempting > to do this for SDL before, but not in a way that worked very well.) > > Signed-off-by: Peter Maydell > --- > In an ideal world, we'd fix the Cocoa UI front end to play nicely > with others, but in the meantime, let's at least give a helpful > message to users if they try to select combinations of configure > options that won't work. > > configure | 20 ++++++++++++++++---- > 1 file changed, 16 insertions(+), 4 deletions(-) Reviewed-by: Daniel P. berrange > > diff --git a/configure b/configure > index cd219d8..826858e 100755 > --- a/configure > +++ b/configure > @@ -956,7 +956,6 @@ for opt do > ;; > --enable-cocoa) > cocoa="yes" ; > - sdl="no" ; > audio_drv_list="coreaudio `echo $audio_drv_list | sed s,coreaudio,,g`" > ;; > --disable-system) softmmu="no" > @@ -1711,6 +1710,21 @@ else > fi > > ########################################## > +# cocoa implies not SDL or GTK > +# (the cocoa UI code currently assumes it is always the active UI > +# and doesn't interact well with other UI frontend code) > +if test "$cocoa" = "yes"; then > + if test "$sdl" = "yes"; then > + error_exit "Cocoa and SDL UIs cannot both be enabled at once" > + fi > + if test "$gtk" = "yes"; then > + error_exit "Cocoa and GTK UIs cannot both be enabled at once" > + fi > + gtk=no > + sdl=no > +fi > + > +########################################## > # L2TPV3 probe > > cat > $TMPC < @@ -2287,9 +2301,7 @@ EOF > if test "$_sdlversion" -lt 121 ; then > sdl_too_old=yes > else > - if test "$cocoa" = "no" ; then > - sdl=yes > - fi > + sdl=yes > fi > > # static link with sdl ? (note: sdl.pc's --static --libs is broken) > -- > 1.9.1 > > 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 :|