Re: [Qemu-devel] [PATCH 1/3] raw-posix: Do not use CONFIG_COCOA macro

2012-05-02 Thread Kevin Wolf
Am 01.05.2012 00:15, schrieb Andreas Färber:
 From: Pavel Borzenkov pavel.borzen...@gmail.com
 
 Use __APPLE__ and __MACH__ macros instead of CONFIG_COCOA to detect Mac
 OS X host. The patch is based on Ben Leslie's patch:
 http://patchwork.ozlabs.org/patch/97859/
 
 Signed-off-by: Ben Leslie be...@benno.id.au
 Signed-off-by: Pavel Borzenkov pavel.borzen...@gmail.com
 Signed-off-by: Andreas Färber andreas.faer...@web.de
 ---
  block/raw-posix.c |8 
  1 files changed, 4 insertions(+), 4 deletions(-)
 
 diff --git a/block/raw-posix.c b/block/raw-posix.c
 index 2d1bc13..03fcfcc 100644
 --- a/block/raw-posix.c
 +++ b/block/raw-posix.c
 @@ -29,7 +29,7 @@
  #include module.h
  #include block/raw-posix-aio.h
  
 -#ifdef CONFIG_COCOA
 +#if defined(__APPLE__)  (__MACH__)

Is there a 'defined' missing?

Kevin



Re: [Qemu-devel] [PATCH 1/3] raw-posix: Do not use CONFIG_COCOA macro

2012-05-02 Thread Andreas Färber
Am 02.05.2012 10:02, schrieb Kevin Wolf:
 Am 01.05.2012 00:15, schrieb Andreas Färber:
 From: Pavel Borzenkov pavel.borzen...@gmail.com

 Use __APPLE__ and __MACH__ macros instead of CONFIG_COCOA to detect Mac
 OS X host. The patch is based on Ben Leslie's patch:
 http://patchwork.ozlabs.org/patch/97859/

 Signed-off-by: Ben Leslie be...@benno.id.au
 Signed-off-by: Pavel Borzenkov pavel.borzen...@gmail.com
 Signed-off-by: Andreas Färber andreas.faer...@web.de
 ---
  block/raw-posix.c |8 
  1 files changed, 4 insertions(+), 4 deletions(-)

 diff --git a/block/raw-posix.c b/block/raw-posix.c
 index 2d1bc13..03fcfcc 100644
 --- a/block/raw-posix.c
 +++ b/block/raw-posix.c
 @@ -29,7 +29,7 @@
  #include module.h
  #include block/raw-posix-aio.h
  
 -#ifdef CONFIG_COCOA
 +#if defined(__APPLE__)  (__MACH__)
 
 Is there a 'defined' missing?

Seems like it, I'll send you a fixup patch.

Thanks for noticing,

Andreas



Re: [Qemu-devel] [PATCH 1/3] raw-posix: Do not use CONFIG_COCOA macro

2012-04-30 Thread Andreas Färber
Am 01.05.2012 00:15, schrieb Andreas Färber:
 From: Pavel Borzenkov pavel.borzen...@gmail.com
 
 Use __APPLE__ and __MACH__ macros instead of CONFIG_COCOA to detect Mac
 OS X host. The patch is based on Ben Leslie's patch:
 http://patchwork.ozlabs.org/patch/97859/
 
 Signed-off-by: Ben Leslie be...@benno.id.au
 Signed-off-by: Pavel Borzenkov pavel.borzen...@gmail.com

Gotta forget something: I fixed up the cocoa-for-upstream branch with
Kevin's Acked-by.

Andreas

 Signed-off-by: Andreas Färber andreas.faer...@web.de
 ---
  block/raw-posix.c |8 
  1 files changed, 4 insertions(+), 4 deletions(-)
 
 diff --git a/block/raw-posix.c b/block/raw-posix.c
 index 2d1bc13..03fcfcc 100644
 --- a/block/raw-posix.c
 +++ b/block/raw-posix.c
 @@ -29,7 +29,7 @@
  #include module.h
  #include block/raw-posix-aio.h
  
 -#ifdef CONFIG_COCOA
 +#if defined(__APPLE__)  (__MACH__)
  #include paths.h
  #include sys/param.h
  #include IOKit/IOKitLib.h
 @@ -505,7 +505,7 @@ again:
  }
  if (size == 0)
  #endif
 -#ifdef CONFIG_COCOA
 +#if defined(__APPLE__)  defined(__MACH__)
  size = LONG_LONG_MAX;
  #else
  size = lseek(fd, 0LL, SEEK_END);
 @@ -650,7 +650,7 @@ static BlockDriver bdrv_file = {
  /***/
  /* host device */
  
 -#ifdef CONFIG_COCOA
 +#if defined(__APPLE__)  defined(__MACH__)
  static kern_return_t FindEjectableCDMedia( io_iterator_t *mediaIterator );
  static kern_return_t GetBSDPath( io_iterator_t mediaIterator, char *bsdPath, 
 CFIndex maxPathSize );
  
 @@ -728,7 +728,7 @@ static int hdev_open(BlockDriverState *bs, const char 
 *filename, int flags)
  {
  BDRVRawState *s = bs-opaque;
  
 -#ifdef CONFIG_COCOA
 +#if defined(__APPLE__)  defined(__MACH__)
  if (strstart(filename, /dev/cdrom, NULL)) {
  kern_return_t kernResult;
  io_iterator_t mediaIterator;