OSS fixes for soundcard.h include path

[marillat]

OSS doesn't build. This patch fix this problem :+1:

--- a/configure
+++ b/configure
@@ -313,8 +313,8 @@ check_oss()

    OSS_CFLAGS=""
    OSS_LIBS=""
-   msg_checking "for header <sys/soundcard.h>"
-   if test -f /usr/include/sys/soundcard.h
+   msg_checking "for header <linux/soundcard.h>"
+   if test -f /usr/include/linux/soundcard.h
    then
        msg_result "yes"
        makefile_vars OSS_CFLAGS OSS_LIBS
--- a/mixer_oss.c
+++ b/mixer_oss.c
@@ -29,7 +29,7 @@
 #if defined(__OpenBSD__)
 #include <soundcard.h>
 #else
-#include <sys/soundcard.h>
+#include <linux/soundcard.h>
 #endif

 enum {
--- a/oss.c
+++ b/oss.c
@@ -24,7 +24,7 @@
 #if defined(__OpenBSD__)
 #include <soundcard.h>
 #else
-#include <sys/soundcard.h>
+#include <linux/soundcard.h>
 #endif
 #include <sys/ioctl.h>
 #include <sys/types.h>

[flyingmutant]

On what system does OSS fail to build?

I believe OSS is primarily useful on non-linux unixes, so hardcoding linux is most probably wrong thing to do.

[marillat]

Linux and kfreebsd.

Some people still use OSS under Linux.

[flyingmutant]

Yeah, sure — but we must be certain that non-linux builds are not broken by this change. Also, I do not see <linux/soundcard.h> anywhere in the OSS documentation.

BTW, what version of OSS do you have installed?

[marillat]

oss4-dev package version 4.2-build2010-4 from Debian unstable