search

croissanne / portaudio_opensles

android opensles hostapi for portaudio
Other
48 stars 17 forks source link

Merge with master #11

Closed darksylinc closed 3 years ago

darksylinc commented 3 years ago

Closes #10

This is a merge with a much more recent commit from upstream.

There was little to merge since most contributions had been isolated to pa_opensles.c and a few more changes for build support.

The old README.txt got replaced with README.md; I moved your changes there. The new README lists the backends and I forgot to include opensles there.

I've been testing Output in the last few days and it seems to be working perfectly. I did not test audio recording.

I only tested building with CMake. To build with CMake simply do:

mkdir build_android
cd build_android
cmake ../ -GNinja -DANDROID_ABI=arm64-v8a -DANDROID_NATIVE_API_LEVEL=24 -DANDROID_NDK=/home/matias/Android/Sdk/ndk/21.0.6113669/ -DCMAKE_TOOLCHAIN_FILE=/home/matias/Android/Sdk/ndk/21.0.6113669/build/cmake/android.toolchain.cmake
ninja

Where /home/matias/Android/Sdk/ndk/21.0.6113669/ is the path to your NDK installation. It will obviously be a different path in your system.

This build instructions probably works on Windows too (except -GNinja should be removed and make should be used).

I noticed there are no instructions on how to build for Android so probably this snippet should be considered for inclusion in the README?

croissanne commented 3 years ago

Thanks! I rebased on top of current portaudio master in https://github.com/Gundersanne/portaudio_opensles/tree/opensles-pr-candidate. I think the latter branch integrates the cmake changes a bit better, that has changed quite a lot since.

However I created #12 and #13 after those + merging the formats branch I might open a PR to portaudio.