Audio library and OpenAL on other platforms

[mosra]

Currently the Audio library is known to work on Windows, Linux and OSX. It compiles for iOS so I assume it works also there. The remaining platforms that aren't tested are:

• Windows + MinGW -- the CI tests are crashing for some reason
• Emscripten -- OpenAL is provided there somehow, but I had no chance to test it.
• Windows RT -- https://github.com/kakashidinho/openal-soft-winphone
• Android -- OpenAL Soft should have a OpenSL ES backend. Some outdated info here: http://stackoverflow.com/a/7506114/6108877

It would be great to investigate the possibilities, document them and enable the Audio library in the CI builds for those platforms.

[Squareys]

@mosra MinGW + Windows:

from https://chocolatey.org/packages/mingw

The default package now uses SJLJ (32-bit) and SEH (64-bit) exception handling and posix threads. Other configurations can be selected using the params:
exception: sjlj, seh (64-bit only), dwarf (32-bit only)
threads: posix, win32
For example: -params "/exception:dwarf /threads:win32"

So you are building 64 bit binaries, but copying Win32/soft_al.dll instead of Win64/soft_al.dll.
Either use 32bit MinGW or copy Win64/soft_al.dll to OpenAL32.dll (yes, 32 is intentional).

[ghost]

@mosra Emscripten + OpenAL is working, but the current FindOpenAL does not work.
Emscripten's has its own OpenAL that works, which I just copy into a directory and point Magnum towards on Emscripten builds

[stevetranby]

Is the Android OpenSL backend working yet?

[Squareys]

@stevetranby Magnum::Audio uses OpenAL. https://github.com/kcat/openal-soft is an implementation of OpenAL which also supports OpenSL backend. Magnum::Audio should work perfectly fine with that.

[mosra]

@alicemargatroid Amazing, thank you! I forked the upstream FindOpenAL.cmake and made modifications in order to make it working on both desktop/mobile systems and Emscripten and now I'm also trying to enable building of Audio library on the CI. It's the openal-emscripten branch, once it goes green, I'll merge the changes to master.

[mosra]

The Audio library is building on Windows CIs for some time already and with 85979ad it is also fully working on the Emscripten and passing tests on the CI.

[ghost]

@mosra OpenAL seems to not compile on Ubuntu 16.06 and some other implementations now; adding

#include <alext.h>

to MagnumIntegration/OpenAL/extensions.h seems to fix it; I'd submit a PR but it is a tiny tiny fix.

[mosra]

Can you paste the exact error? I vaguely remember that alext.h was not available everywhere and by having my own OpenAL/extensions.h I aimed to fix that.

[ghost]

The exact error is that most of the new extensions (AL_FORMAT_REAR16, etc etc) aren't found. Maybe we could just copy and paste their defines into OpenAL/extensions.h?

…

On Tue, Jan 3, 2017 at 3:27 AM, Vladimír Vondruš ***@***.***> wrote: Can you paste the exact error? I vaguely remember that alext.h was not available everywhere and by having my own OpenAL/extensions.h I aimed to fix that. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#149 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACvGBuCvsjoooyZuw9uOyY9-paZORcwQks5rOgZpgaJpZM4IuOkq> .

[mosra]

Huh... but they are there, you added them in 1897e36:

magnum/src/MagnumExternal/OpenAL/extensions.h

Line 76 in 0875093

#define AL_FORMAT_REAR16 0x1208

...

/* AL_EXT_MCFORMATS */
#ifndef AL_EXT_MCFORMATS
#define AL_EXT_MCFORMATS 1
#define AL_FORMAT_QUAD8                          0x1204
#define AL_FORMAT_QUAD16                         0x1205
#define AL_FORMAT_QUAD32                         0x1206
#define AL_FORMAT_REAR8                          0x1207
#define AL_FORMAT_REAR16                         0x1208
#define AL_FORMAT_REAR32                         0x1209
#define AL_FORMAT_51CHN8                         0x120A

...

Might this be because of some header conflict?

[ghost]

I'll check when I get home. Maybe I'm getting a weird conflict, my apologies if it is on my end.

…

On Tue, Jan 3, 2017 at 9:19 AM, Vladimír Vondruš ***@***.***> wrote: Huh... but they *are* there: https://github.com/mosra/magnum/blob/ 0875093/src/MagnumExternal/ OpenAL/extensions.h#L76 ... /* AL_EXT_MCFORMATS */ #ifndef AL_EXT_MCFORMATS #define AL_EXT_MCFORMATS 1 #define AL_FORMAT_QUAD8 0x1204 #define AL_FORMAT_QUAD16 0x1205 #define AL_FORMAT_QUAD32 0x1206 #define AL_FORMAT_REAR8 0x1207 #define AL_FORMAT_REAR16 0x1208 #define AL_FORMAT_REAR32 0x1209 #define AL_FORMAT_51CHN8 0x120A ... Might this be because of some header conflict? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#149 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACvGBl0pLGYXQxPL4KoXOfe5xLg90gUDks5rOljlgaJpZM4IuOkq> .