Re: [Qemu-devel] Add option to disable Cocoa on Mac OS X

2011-05-29 Thread Andreas Färber

Am 16.05.2011 um 12:57 schrieb Ben Leslie:

On Wed, May 11, 2011 at 23:54, Peter Maydell  
peter.mayd...@linaro.org wrote:

On 7 May 2011 12:40, Alexander Graf ag...@suse.de wrote:
So I suppose the only thing missing is a --disable-cocoa option,  
yup.


I've just noticed that some of the code in block/raw-posix.c
uses the CONFIG_COCOA #define to gate whether to do MacOSX
specific handling of CDROMs and so on. I'm not a MacOS expert
but maybe that needs to be changed to some other ifdef -- I'm
guessing we don't want to have cdrom handling randomly change
behaviour just because the user tried to disable Cocoa graphics
handling...


These seems to be a good point. Potentially checking for __APPLE__ and
__MACH__ makes more sense? (See: http://predef.sourceforge.net/preos.html#sec20)


Any update on this?

IIUC without digging further, today, Cocoa not only means displaying  
the graphical window but also has impact on which libraries are linked  
to, such as IOKit and CoreAudio. So simply replacing CONFIG_COCOA with  
__APPLE__ would seem not to be enough.


Tristan, if your patch takes care of everything, mind to resend or to  
provide a Patchwork link please?


Andreas



Re: [Qemu-devel] Add option to disable Cocoa on Mac OS X

2011-05-16 Thread Ben Leslie
On Wed, May 11, 2011 at 23:54, Peter Maydell peter.mayd...@linaro.org wrote:
 On 7 May 2011 12:40, Alexander Graf ag...@suse.de wrote:
 So I suppose the only thing missing is a --disable-cocoa option, yup.

 I've just noticed that some of the code in block/raw-posix.c
 uses the CONFIG_COCOA #define to gate whether to do MacOSX
 specific handling of CDROMs and so on. I'm not a MacOS expert
 but maybe that needs to be changed to some other ifdef -- I'm
 guessing we don't want to have cdrom handling randomly change
 behaviour just because the user tried to disable Cocoa graphics
 handling...

These seems to be a good point. Potentially checking for __APPLE__ and
__MACH__ makes more sense? (See: http://predef.sourceforge.net/preos.html#sec20)

Cheers,

Ben



Re: [Qemu-devel] Add option to disable Cocoa on Mac OS X

2011-05-11 Thread Peter Maydell
On 7 May 2011 12:40, Alexander Graf ag...@suse.de wrote:
 So I suppose the only thing missing is a --disable-cocoa option, yup.

I've just noticed that some of the code in block/raw-posix.c
uses the CONFIG_COCOA #define to gate whether to do MacOSX
specific handling of CDROMs and so on. I'm not a MacOS expert
but maybe that needs to be changed to some other ifdef -- I'm
guessing we don't want to have cdrom handling randomly change
behaviour just because the user tried to disable Cocoa graphics
handling...

-- PMM



Re: [Qemu-devel] Add option to disable Cocoa on Mac OS X

2011-05-07 Thread Andreas Färber

Hi,

Am 06.05.2011 um 14:01 schrieb Alexander Graf:


On 06.05.2011, at 12:46, Tristan Gingold wrote:


On May 6, 2011, at 11:48 AM, Ben Leslie wrote:

Are there any objections to adding a --disable-cocoa configure  
option?
For simulating ARM microcontrollers I have no desire or need for  
graphics.


Seconded.  I think I have once posted such a patch.


Not aware, sorry.


Andreas?


My build scripts used to need an explicit --enable-cocoa.
Did the default somehow change so that --disable-cocoa is necessary  
now to disable it? Or is it just for symmetry?


Andreas



Re: [Qemu-devel] Add option to disable Cocoa on Mac OS X

2011-05-07 Thread Alexander Graf

On 07.05.2011, at 12:12, Andreas Färber wrote:

 Hi,
 
 Am 06.05.2011 um 14:01 schrieb Alexander Graf:
 
 On 06.05.2011, at 12:46, Tristan Gingold wrote:
 
 On May 6, 2011, at 11:48 AM, Ben Leslie wrote:
 
 Are there any objections to adding a --disable-cocoa configure option?
 For simulating ARM microcontrollers I have no desire or need for graphics.
 
 Seconded.  I think I have once posted such a patch.
 
 Not aware, sorry.
 
 Andreas?
 
 My build scripts used to need an explicit --enable-cocoa.
 Did the default somehow change so that --disable-cocoa is necessary now to 
 disable it? Or is it just for symmetry?

It definitely gets enabled by default:

 dekai:qemu alex$ ./configure --target-list=x86_64-softmmu
 Install prefix/usr/local
 BIOS directory/usr/local/share/qemu
 binary directory  /usr/local/bin
 config directory  /usr/local/etc
 Manual directory  /usr/local/share/man
 ELF interp prefix /usr/gnemul/qemu-%M
 Source path   /Users/alex/git/qemu
 C compilergcc
 Host C compiler   gcc
 CFLAGS-O2 -g 
 QEMU_CFLAGS   -m64 -arch x86_64 -D_FORTIFY_SOURCE=2 -D_GNU_SOURCE 
 -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wstrict-prototypes 
 -Wredundant-decls -Wall -Wundef -Wendif-labels -Wwrite-strings 
 -Wmissing-prototypes -fno-strict-aliasing  -fstack-protector-all 
 -Wmissing-include-dirs -Wempty-body -Wnested-externs -Wformat-security 
 -Wformat-y2k -Winit-self -Wold-style-definition
 LDFLAGS   -m64 -framework CoreFoundation -framework IOKit -arch 
 x86_64 -g 
 make  make
 install   install
 host CPU  x86_64
 host big endian   no
 target list   x86_64-softmmu
 tcg debug enabled no
 Mon debug enabled no
 gprof enabled no
 sparse enabledno
 strip binariesyes
 profiler  no
 static build  no
 -Werror enabled   no
 Cocoa support yes

So I suppose the only thing missing is a --disable-cocoa option, yup.


Alex




[Qemu-devel] Add option to disable Cocoa on Mac OS X

2011-05-06 Thread Ben Leslie
Hi all,

Are there any objections to adding a --disable-cocoa configure option?
For simulating ARM microcontrollers I have no desire or need for graphics.

Thanks,

Benno



Re: [Qemu-devel] Add option to disable Cocoa on Mac OS X

2011-05-06 Thread Tristan Gingold

On May 6, 2011, at 11:48 AM, Ben Leslie wrote:

 Hi all,
 
 Are there any objections to adding a --disable-cocoa configure option?
 For simulating ARM microcontrollers I have no desire or need for graphics.

Seconded.  I think I have once posted such a patch.

Tristan.




Re: [Qemu-devel] Add option to disable Cocoa on Mac OS X

2011-05-06 Thread Alexander Graf

On 06.05.2011, at 12:46, Tristan Gingold wrote:

 
 On May 6, 2011, at 11:48 AM, Ben Leslie wrote:
 
 Hi all,
 
 Are there any objections to adding a --disable-cocoa configure option?
 For simulating ARM microcontrollers I have no desire or need for graphics.
 
 Seconded.  I think I have once posted such a patch.

Andreas?


Alex