From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:47428) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rn6to-0001fD-3t for qemu-devel@nongnu.org; Tue, 17 Jan 2012 06:08:25 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Rn6tj-0000u7-Iv for qemu-devel@nongnu.org; Tue, 17 Jan 2012 06:08:20 -0500 Received: from mout.web.de ([212.227.15.3]:58110) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rn6tj-0000tr-6q for qemu-devel@nongnu.org; Tue, 17 Jan 2012 06:08:15 -0500 Message-ID: <4F155626.8050200@web.de> Date: Tue, 17 Jan 2012 12:06:14 +0100 From: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= MIME-Version: 1.0 References: <1320950407-43512-1-git-send-email-pavel.borzenkov@gmail.com> <1320950407-43512-4-git-send-email-pavel.borzenkov@gmail.com> <4EE007C5.4060109@web.de> <4F10CF6B.20008@web.de> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH 3/3] configure: add '--disable-cocoa' switch List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Andrzej Zaborowski Cc: Rui Carmo , Pavel Borzenkov , Ben Leslie , qemu-devel@nongnu.org Am 17.01.2012 00:46, schrieb Andrzej Zaborowski: > On 14 January 2012 01:42, Andreas Färber wrote: >> Am 08.12.2011 01:41, schrieb Andreas Färber: >>> Am 10.11.2011 19:40, schrieb Pavel Borzenkov: >>>> When SDL support is disabled, there is no way to build QEMU without >>>> Cocoa support on MacOS X. This patch adds '--disable-cocoa' switch and >>>> allows to build QEMU without both SDL and Cocoa frontends. >>>> >>>> Signed-off-by: Pavel Borzenkov >>>> --- >>>> configure | 7 ++++++- >>>> 1 files changed, 6 insertions(+), 1 deletions(-) >>>> >>>> diff --git a/configure b/configure >>>> index 401d9a6..4720bb2 100755 >>>> --- a/configure >>>> +++ b/configure >>>> @@ -670,6 +670,8 @@ for opt do >>>> ;; >>>> --enable-profiler) profiler="yes" >>>> ;; >>>> + --disable-cocoa) cocoa="no" >>>> + ;; >>>> --enable-cocoa) >>>> cocoa="yes" ; >>>> sdl="no" ; >>> >>> Tested-by: Andreas Färber >>> >>>> @@ -980,7 +982,10 @@ echo " --disable-sdl disable SDL" >>>> echo " --enable-sdl enable SDL" >>>> echo " --disable-vnc disable VNC" >>>> echo " --enable-vnc enable VNC" >>>> -echo " --enable-cocoa enable COCOA (Mac OS X only)" >>>> +if test "$darwin" = "yes" ; then >>>> + echo " --disable-cocoa disable COCOA" >>>> + echo " --enable-cocoa enable COCOA (default)" >>>> +fi >>>> echo " --audio-drv-list=LIST set audio drivers list:" >>>> echo " Available drivers: $audio_possible_drivers" >>>> echo " --audio-card-list=LIST set list of emulated audio cards [$audio_card_list]" >>> >>> I see no prior art of any conditional help output in configure. Anthony? >>> Andrzej? >> >> Ping? Should we keep command line options a flat list with comments on >> applicability or start introducing tests like above? >> >> Me, I'd prefer not doing this since the switch cases above don't check. > > Perhaps --diable-cocoa should be allowed on any platform. You're > right we don't have such checks now, but then it's hard to see > downsides of doing them, so I'm quite ambivalent. > > I still don't see the purpose of the following test in configure though: > if test "$cocoa" = "no" ; then > sdl=yes > fi The reason is that SDL and Cocoa must not both be detected due to the way it is handled in vl.c. Given that Cocoa is enabled by default, SDL must not be detected in that case. If Cocoa is disabled, then detecting SDL is fine of course. The alternative would be to check for SDL and, if present, disable Cocoa. Still collides with Pavel's patch though - we need to sort out how to merge them. > With it in place and no --disable-cocoa there's no way to compile SDL. > --enable-cocoa is also broken as fas as I can tell. I've been using it for historic reasons without noticeable problems. Andreas