From 21fb1b461c29b61faf199ec2362ff9a53d6f01e6 Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Mon, 17 Jan 2022 21:17:54 -0500 Subject: [PATCH] newlib: move version defines out of the config headers This will make it easier to move newlib.h to use autoheader directly. We only want the newlib version defines in our hand curated version file, _newlib_version.h, not in the template header, newlib.h, so using AC_DEFINE doesn't make much sense. --- newlib/Makefile.in | 4 ++++ newlib/_newlib_version.hin | 16 +++++++++++----- newlib/configure | 16 ++++++---------- newlib/configure.ac | 10 ++++++---- 4 files changed, 27 insertions(+), 19 deletions(-) diff --git a/newlib/Makefile.in b/newlib/Makefile.in index 1af918059..267db9019 100644 --- a/newlib/Makefile.in +++ b/newlib/Makefile.in @@ -315,6 +315,10 @@ MAINT = @MAINT@ MAKEINFO = @MAKEINFO@ MKDIR_P = @MKDIR_P@ NEWLIB_CFLAGS = @NEWLIB_CFLAGS@ +NEWLIB_MAJOR_VERSION = @NEWLIB_MAJOR_VERSION@ +NEWLIB_MINOR_VERSION = @NEWLIB_MINOR_VERSION@ +NEWLIB_PATCHLEVEL_VERSION = @NEWLIB_PATCHLEVEL_VERSION@ +NEWLIB_VERSION = @NEWLIB_VERSION@ NM = @NM@ NMEDIT = @NMEDIT@ NO_INCLUDE_LIST = @NO_INCLUDE_LIST@ diff --git a/newlib/_newlib_version.hin b/newlib/_newlib_version.hin index 03d4e3d47..077b81ecf 100644 --- a/newlib/_newlib_version.hin +++ b/newlib/_newlib_version.hin @@ -2,10 +2,16 @@ #ifndef _NEWLIB_VERSION_H__ #define _NEWLIB_VERSION_H__ 1 -#undef _NEWLIB_VERSION -#undef __NEWLIB__ -#undef __NEWLIB_MINOR__ -#undef __NEWLIB_PATCHLEVEL__ +/* The newlib version in string format. */ +#define _NEWLIB_VERSION "@NEWLIB_VERSION@" -#endif /* !_NEWLIB_VERSION_H__ */ +/* The newlib major version number. */ +#define __NEWLIB__ @NEWLIB_MAJOR_VERSION@ + +/* The newlib minor version number. */ +#define __NEWLIB_MINOR__ @NEWLIB_MINOR_VERSION@ +/* The newlib patch level. */ +#define __NEWLIB_PATCHLEVEL__ @NEWLIB_PATCHLEVEL_VERSION@ + +#endif /* !_NEWLIB_VERSION_H__ */ diff --git a/newlib/configure b/newlib/configure index 624ec89af..2be0a6c33 100755 --- a/newlib/configure +++ b/newlib/configure @@ -638,6 +638,10 @@ HAVE_LONG_DOUBLE_FALSE HAVE_LONG_DOUBLE_TRUE libc_cv_cc_loop_to_function libc_cv_initfinit_array +NEWLIB_PATCHLEVEL_VERSION +NEWLIB_MINOR_VERSION +NEWLIB_MAJOR_VERSION +NEWLIB_VERSION EXEEXT_FOR_BUILD CC_FOR_BUILD STDIO64_OBJECTLIST @@ -11970,7 +11974,7 @@ else lt_dlunknown=0; lt_dlno_uscore=1; lt_dlneed_uscore=2 lt_status=$lt_dlunknown cat > conftest.$ac_ext <<_LT_EOF -#line 11973 "configure" +#line 11977 "configure" #include "confdefs.h" #if HAVE_DLFCN_H @@ -12076,7 +12080,7 @@ else lt_dlunknown=0; lt_dlno_uscore=1; lt_dlneed_uscore=2 lt_status=$lt_dlunknown cat > conftest.$ac_ext <<_LT_EOF -#line 12079 "configure" +#line 12083 "configure" #include "confdefs.h" #if HAVE_DLFCN_H @@ -12572,16 +12576,8 @@ _ACEOF -$as_echo "#define _NEWLIB_VERSION \"4.2.0\"" >>confdefs.h -$as_echo "#define __NEWLIB__ 4" >>confdefs.h - - -$as_echo "#define __NEWLIB_MINOR__ 2" >>confdefs.h - - -$as_echo "#define __NEWLIB_PATCHLEVEL__ 0" >>confdefs.h if test "${multilib}" = "yes"; then diff --git a/newlib/configure.ac b/newlib/configure.ac index 35d94666a..e1fb2ad0e 100644 --- a/newlib/configure.ac +++ b/newlib/configure.ac @@ -477,10 +477,12 @@ if test "${newlib_mb}" = "yes"; then fi AC_DEFINE_UNQUOTED(_MB_LEN_MAX, $_mb_len_max, [Multibyte max length.]) -AC_DEFINE(_NEWLIB_VERSION, "NEWLIB_VERSION", [The newlib version in string format.]) -AC_DEFINE(__NEWLIB__, NEWLIB_MAJOR_VERSION, [The newlib major version number.]) -AC_DEFINE(__NEWLIB_MINOR__, NEWLIB_MINOR_VERSION, [The newlib minor version number.]) -AC_DEFINE(__NEWLIB_PATCHLEVEL__, NEWLIB_PATCHLEVEL_VERSION, [The newlib patch level.]) +dnl These are AC_SUBST instead of AC_DEFINE as they're hand maintained in a +dnl dedicated _newlib_version.h, and we don't want them in newlib.h. +AC_SUBST([NEWLIB_VERSION]) +AC_SUBST([NEWLIB_MAJOR_VERSION]) +AC_SUBST([NEWLIB_MINOR_VERSION]) +AC_SUBST([NEWLIB_PATCHLEVEL_VERSION]) if test "${multilib}" = "yes"; then multilib_arg="--enable-multilib" -- 2.43.5