Hi,
both our arm64 and armv7 require at least NEON support. audio/openal
checks for NEON support by opening /proc/cpuinfo and parsing it, which
we obviously don't have. I'd propose just patching it and enabling it
for OpenBSD. At the very least, it gets rid of this obnoxious message:
Failed to open /proc/cpuinfo, cannot check for NEON support
Tested by running games/openal.
Opinions?
Patrick
diff --git a/audio/openal/Makefile b/audio/openal/Makefile
index 871aa7bbf6c..a1de0e0467b 100644
--- a/audio/openal/Makefile
+++ b/audio/openal/Makefile
@@ -7,6 +7,7 @@ EPOCH = 0
DISTNAME = openal-soft-$V
PKGNAME = openal-$V
CATEGORIES = audio
+REVISION = 0
SHARED_LIBS = openal 4.0
diff --git a/audio/openal/patches/patch-alc_helpers_cpp
b/audio/openal/patches/patch-alc_helpers_cpp
new file mode 100644
index 00000000000..f3f781d5afc
--- /dev/null
+++ b/audio/openal/patches/patch-alc_helpers_cpp
@@ -0,0 +1,23 @@
+$OpenBSD$
+
+Index: alc/helpers.cpp
+--- alc/helpers.cpp.orig
++++ alc/helpers.cpp
+@@ -161,6 +161,9 @@ void FillCPUCaps(int capfilter)
+ #endif
+ #endif
+ #ifdef HAVE_NEON
++#if defined(__OpenBSD__)
++ caps |= CPU_CAP_NEON;
++#else
+ al::ifstream file{"/proc/cpuinfo"};
+ if(!file.is_open())
+ ERR("Failed to open /proc/cpuinfo, cannot check for NEON support\n");
+@@ -193,6 +196,7 @@ void FillCPUCaps(int capfilter)
+ }
+ }
+ }
++#endif
+ #endif
+
+ TRACE("Extensions:%s%s%s%s%s%s\n",
