All of lore.kernel.org
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] Remove OSS support for OpenBSD
@ 2013-05-24 23:01 Brad Smith
  2013-05-24 23:50 ` Laszlo Ersek
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Brad Smith @ 2013-05-24 23:01 UTC (permalink / raw)
  To: qemu-devel

Remove the OSS support for OpenBSD. The OSS API has not been usable
for quite some time.

Signed-off-by: Brad Smith <brad@comstyle.com>


diff --git a/audio/ossaudio.c b/audio/ossaudio.c
index 00be9c9..007c641 100644
--- a/audio/ossaudio.c
+++ b/audio/ossaudio.c
@@ -25,11 +25,7 @@
 #include <sys/mman.h>
 #include <sys/types.h>
 #include <sys/ioctl.h>
-#ifdef __OpenBSD__
-#include <soundcard.h>
-#else
 #include <sys/soundcard.h>
-#endif
 #include "qemu-common.h"
 #include "qemu/main-loop.h"
 #include "qemu/host-utils.h"
diff --git a/configure b/configure
index 5ae7e4a..eb74510 100755
--- a/configure
+++ b/configure
@@ -468,9 +468,8 @@ NetBSD)
 OpenBSD)
   bsd="yes"
   make="${MAKE-gmake}"
-  audio_drv_list="oss"
-  audio_possible_drivers="oss sdl esd"
-  oss_lib="-lossaudio"
+  audio_drv_list="sdl"
+  audio_possible_drivers="sdl esd"
 ;;
 Darwin)
   bsd="yes"

-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] [PATCH] Remove OSS support for OpenBSD
  2013-05-24 23:01 [Qemu-devel] [PATCH] Remove OSS support for OpenBSD Brad Smith
@ 2013-05-24 23:50 ` Laszlo Ersek
  2013-05-25 19:12   ` Andreas Färber
  2013-05-26 17:04 ` Blue Swirl
  2 siblings, 0 replies; 5+ messages in thread
From: Laszlo Ersek @ 2013-05-24 23:50 UTC (permalink / raw)
  To: Brad Smith; +Cc: qemu-devel

On 05/25/13 01:01, Brad Smith wrote:
> Remove the OSS support for OpenBSD. The OSS API has not been usable
> for quite some time.
> 
> Signed-off-by: Brad Smith <brad@comstyle.com>
> 
> 
> diff --git a/audio/ossaudio.c b/audio/ossaudio.c
> index 00be9c9..007c641 100644
> --- a/audio/ossaudio.c
> +++ b/audio/ossaudio.c
> @@ -25,11 +25,7 @@
>  #include <sys/mman.h>
>  #include <sys/types.h>
>  #include <sys/ioctl.h>
> -#ifdef __OpenBSD__
> -#include <soundcard.h>
> -#else
>  #include <sys/soundcard.h>
> -#endif
>  #include "qemu-common.h"
>  #include "qemu/main-loop.h"
>  #include "qemu/host-utils.h"
> diff --git a/configure b/configure
> index 5ae7e4a..eb74510 100755
> --- a/configure
> +++ b/configure
> @@ -468,9 +468,8 @@ NetBSD)
>  OpenBSD)
>    bsd="yes"
>    make="${MAKE-gmake}"
> -  audio_drv_list="oss"
> -  audio_possible_drivers="oss sdl esd"
> -  oss_lib="-lossaudio"
> +  audio_drv_list="sdl"
> +  audio_possible_drivers="sdl esd"
>  ;;
>  Darwin)
>    bsd="yes"
> 

Reviewed-by: Laszlo Ersek <lersek@redhat.com>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Qemu-trivial] [Qemu-devel] [PATCH] Remove OSS support for OpenBSD
  2013-05-24 23:01 [Qemu-devel] [PATCH] Remove OSS support for OpenBSD Brad Smith
@ 2013-05-25 19:12   ` Andreas Färber
  2013-05-25 19:12   ` Andreas Färber
  2013-05-26 17:04 ` Blue Swirl
  2 siblings, 0 replies; 5+ messages in thread
From: Andreas Färber @ 2013-05-25 19:12 UTC (permalink / raw)
  To: Brad Smith; +Cc: qemu-trivial, qemu-devel

Am 25.05.2013 01:01, schrieb Brad Smith:
> Remove the OSS support for OpenBSD. The OSS API has not been usable
> for quite some time.
> 
> Signed-off-by: Brad Smith <brad@comstyle.com>

Reviewed-by: Andreas Färber <afaerber@suse.de>

CC'ing qemu-trivial.

Andreas

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] [PATCH] Remove OSS support for OpenBSD
@ 2013-05-25 19:12   ` Andreas Färber
  0 siblings, 0 replies; 5+ messages in thread
From: Andreas Färber @ 2013-05-25 19:12 UTC (permalink / raw)
  To: Brad Smith; +Cc: qemu-trivial, qemu-devel

Am 25.05.2013 01:01, schrieb Brad Smith:
> Remove the OSS support for OpenBSD. The OSS API has not been usable
> for quite some time.
> 
> Signed-off-by: Brad Smith <brad@comstyle.com>

Reviewed-by: Andreas Färber <afaerber@suse.de>

CC'ing qemu-trivial.

Andreas

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] [PATCH] Remove OSS support for OpenBSD
  2013-05-24 23:01 [Qemu-devel] [PATCH] Remove OSS support for OpenBSD Brad Smith
  2013-05-24 23:50 ` Laszlo Ersek
  2013-05-25 19:12   ` Andreas Färber
@ 2013-05-26 17:04 ` Blue Swirl
  2 siblings, 0 replies; 5+ messages in thread
From: Blue Swirl @ 2013-05-26 17:04 UTC (permalink / raw)
  To: Brad Smith; +Cc: qemu-devel

On Fri, May 24, 2013 at 11:01 PM, Brad Smith <brad@comstyle.com> wrote:
> Remove the OSS support for OpenBSD. The OSS API has not been usable
> for quite some time.
>
> Signed-off-by: Brad Smith <brad@comstyle.com>

Thanks, applied.

>
>
> diff --git a/audio/ossaudio.c b/audio/ossaudio.c
> index 00be9c9..007c641 100644
> --- a/audio/ossaudio.c
> +++ b/audio/ossaudio.c
> @@ -25,11 +25,7 @@
>  #include <sys/mman.h>
>  #include <sys/types.h>
>  #include <sys/ioctl.h>
> -#ifdef __OpenBSD__
> -#include <soundcard.h>
> -#else
>  #include <sys/soundcard.h>
> -#endif
>  #include "qemu-common.h"
>  #include "qemu/main-loop.h"
>  #include "qemu/host-utils.h"
> diff --git a/configure b/configure
> index 5ae7e4a..eb74510 100755
> --- a/configure
> +++ b/configure
> @@ -468,9 +468,8 @@ NetBSD)
>  OpenBSD)
>    bsd="yes"
>    make="${MAKE-gmake}"
> -  audio_drv_list="oss"
> -  audio_possible_drivers="oss sdl esd"
> -  oss_lib="-lossaudio"
> +  audio_drv_list="sdl"
> +  audio_possible_drivers="sdl esd"
>  ;;
>  Darwin)
>    bsd="yes"
>
> --
> This message has been scanned for viruses and
> dangerous content by MailScanner, and is
> believed to be clean.
>
>

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2013-05-26 17:04 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-05-24 23:01 [Qemu-devel] [PATCH] Remove OSS support for OpenBSD Brad Smith
2013-05-24 23:50 ` Laszlo Ersek
2013-05-25 19:12 ` [Qemu-trivial] " Andreas Färber
2013-05-25 19:12   ` Andreas Färber
2013-05-26 17:04 ` Blue Swirl

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.