[pulseaudio-discuss] [PATCH] build-sys: Add an Android-friendly build system

Tanu Kaskinen tanu.kaskinen at linux.intel.com
Wed Jun 25 05:23:39 PDT 2014


On Sun, 2013-09-29 at 17:52 +0530, Arun Raghavan wrote:
> Some notes:
> 
> * This depends on 'androgenizer', a tool to generate Android-style
>   Android.mk files from an autotools build system.
> 
> * This assumes that PA is run as the system daemon and configures
>   accordingly.
> 
> * In the Android build, libltdl is likely not available when configure
>   is being run, and we know it will be so the check is manually
>   overridden.
> 
> * NEON support needs to be communicated from an upper-level Android.mk.
> 
> * rpaths don't work on Android - you need LD_LIBRARY_PATH set.
> 
> * Configuration files for /etc/pulse are shipped out of tree, since they
>   vary from device to device.
> 
> * The original patch was massive and ugly, but thanks Pekka Paalanen's
>   work on Weston to break out common androgenizer snippets, we could
>   greatly decrease the size of the Android-specific elements.
> 
> * Since we don't use libtool for linking, we need to work around some
>   symbol definition problems that turn up. This is done in a separate
>   file (pulseaudio-android-symdef.c). More information about this at:
>   http://www.sourceware.org/autobook/autobook/autobook_173.html (the
>   example snippet needs some fixing to work with current libtool).
> ---
>  Android.mk                             |  94 +++++++++++++++++++++++
>  src/Makefile.am                        | 135 +++++++++++++++++++++++++++++++++
>  src/daemon/pulseaudio-android.symdef.c |   6 ++
>  3 files changed, 235 insertions(+)
>  create mode 100644 Android.mk
>  create mode 100644 src/daemon/pulseaudio-android.symdef.c

I didn't do thorough review, but I'm OK with merging this. One wish,
though: move the Android stuff out from src/Makefile.am to a separate
file, that is then included by src/Makefile.am. That way I don't need to
look at the black magic when I operate on src/Makefile.am.

-- 
Tanu



More information about the pulseaudio-discuss mailing list