[Mesa-dev] [PATCH 1/2] configure.ac: Add support for Android builds

[Emil Velikov]

Hi Nicolas,

On 28 April 2016 at 11:41, Nicolas Boichat <drinkcat at chromium.org> wrote:
> Add support for EGL android platform.
>
> Also, detect when --host finishes with -android. In that case, we
> do not set _GNU_SOURCE, and define autoconf symbol HAVE_ANDROID, so
> that Android-specific workarounds can be applied.
>
I'm quite happy to see patches from Android/CrOS people although I'm a
bit confused here.
Are these are based for CrOS or Android ? I thought Android had it's
own build system.

Is there any documentation on how one can get things going on your platform ?

> Signed-off-by: Nicolas Boichat <drinkcat at google.com>
> ---
>  configure.ac        | 10 ++++++++++
>  src/egl/Makefile.am |  5 +++++
>  2 files changed, 15 insertions(+)
>
> diff --git a/configure.ac b/configure.ac
> index 5f75c60..6ee978c 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -248,7 +248,11 @@ _SAVE_CPPFLAGS="$CPPFLAGS"
>  dnl Compiler macros
>  DEFINES="-D__STDC_LIMIT_MACROS -D__STDC_CONSTANT_MACROS"
>  AC_SUBST([DEFINES])
> +android=no
>  case "$host_os" in
> +*-android)
> +    android=yes
> +    ;;
>  linux*|*-gnu*|gnu*)
>      DEFINES="$DEFINES -D_GNU_SOURCE"
>      ;;
> @@ -260,6 +264,8 @@ cygwin*)
>      ;;
>  esac
>
> +AM_CONDITIONAL(HAVE_ANDROID, test "x$android" = xyes)
> +
>  dnl Add flags for gcc and g++
>  if test "x$GCC" = xyes; then
>      CFLAGS="$CFLAGS -Wall"
> @@ -1959,6 +1965,9 @@ for plat in $egl_platforms; do
>                         AC_MSG_ERROR([EGL platform surfaceless requires libdrm >= $LIBDRM_REQUIRED])
>                 ;;
>
> +       android)
Can you update the documentation in docs/egl.html as well please. Feel
free to do that as follow-up patch.

Do you guys have X related libraries on the host_os ? Or you
explicitly disable those mesa features that depend on it ?

Thanks
Emil