Re: [PATCH 3/5] libgloss: merge libgloss into top-level Makefile

Mike Frysinger <[email protected]> Mon, 7 Feb 2022 06:14:02 -0500

Newsgroups	gmane.comp.lib.newlib
Message-ID	<YgD++hrxLboZ8G1H@vapier>

On 05 Feb 2022 00:46, Mike Frysinger wrote:
> +if CONFIG_LIBNOSYS
> +include libnosys/Makefile.inc
> +endif
> --- a/libgloss/configure.ac
> +++ b/libgloss/configure.ac
> @@ -230,10 +230,7 @@ dnl if test "${config_testsuite}" = "true";
>  dnl   then AC_CONFIG_SUBDIRS([testsuite])
>  dnl fi
>  
> -if test "${config_libnosys}" = "true"; then
> -  AC_CONFIG_FILES([libnosys/Makefile])
> -  subdirs="$subdirs libnosys"
> -fi
> +AM_CONDITIONAL(CONFIG_LIBNOSYS, test x$config_libnosys = xtrue)
>  
>  LIB_AC_PROG_CC
>  AS=${AS-as}

i'm not super happy with the CONFIG_xxx naming.  newlib doesn't seem to have
an existing convention for me to copy & paste.  there is LIB*_MACHINE_DIR,
but those correspond to specific machine/xxx/ subdirs.

i guess i could use the newlib pattern and name these HAVE_xxx_DIR.
-mike

signature.asc (application/pgp-signature, 833 B)