Re: Android port

[Eli Zaretskii]

> Date: Mon, 07 Aug 2023 14:19:57 +0300
> From: Eli Zaretskii <eliz@gnu.org>
> Cc: eggert@cs.ucla.edu, angelo.g0@libero.it, emacs-devel@gnu.org,
>  bruno@clisp.org
> 
> Thanks.  It looks like nt/gnulib-cfg.mk still names modules that were
> eventually removed?  Could you please audit the modules added to
> gnulib-cfg.mk and remove the lines that are no longer needed on
> master?

Also, it looks like neither ENABLE_CHECKING nor GLYPH_DEBUG are being
defined in src/config.h in the non-Android builds, although I passed
the --enable-checking-'yes,glyphs' option to the configure script.
This has something to do with this snippet of configure.ac:

  AS_IF([test "x$XCONFIGURE" = "xandroid" \
         && test "x$android_enable_checking" = "xyes"],
    [ac_enable_checking=yes])

  # There is little point in enabling checking in the build machine if
  # cross-compiling for Android.
  AS_IF([test -z "$with_android" || test -n "$XCONFIGURE"],[
    if test x$ac_enable_checking != x ; then
      AC_DEFINE([ENABLE_CHECKING], [1],
    [Define to 1 if expensive run-time data type and consistency checks are 
enabled.])
    fi

It generates the following code in configure:

  # This environment variable is used to signal that checking should be
  # enabled on Android.  When that happens, simply enable checking for
  # the cross-compiled Android binary.

  if test "x$XCONFIGURE" = "xandroid" \
         && test "x$android_enable_checking" = "xyes"; then :  <<<<<<<<<<<<
    ac_enable_checking=yes
  fi

  # There is little point in enabling checking in the build machine if
  # cross-compiling for Android.
  if test -z "$with_android" || test -n "$XCONFIGURE"; then :  <<<<<<<<<<<<

    if test x$ac_enable_checking != x ; then

  $as_echo "#define ENABLE_CHECKING 1" >>confdefs.h

    fi

and I believe that those colons ':' after 'then' are the culprit.  But
my Autoconf-fu is not strong enough to see what has to be done to fix
this.