Re: [pushed] gdb: fix typos in configure

[Tom de Vries via Gdb-patches <gdb-patches@sourceware.org>]

On 11/27/21 8:34 PM, Simon Marchi via Gdb-patches wrote:
> The variable names used to restore CFLAGS and LDFLAGS here don't quite
> match the names used above, resulting in losing the original CFLAGS and
> LDFLAGS.  Fix that.
> 

Hi Simon,

thanks for fixing this mistake in my patch.

Trying to understand what I could have done to catch his, I tried
shellcheck:
...
$ shellcheck src/gdb/configure 2>&1 | grep -i save
      save_CFLAGS=$CFLAGS
      ^---------^ SC2034: save_CFLAGS appears unused. Verify use (or
export if used externally).
      save_LDFLAGS=$LDFLAGS
      ^----------^ SC2034: save_LDFLAGS appears unused. Verify use (or
export if used externally).
      CFLAGS="$SAVE_CFLAGS"
              ^----------^ SC2153: Possible misspelling: SAVE_CFLAGS may
not be assigned, but save_CFLAGS is.
...
and indeed the problem was found.

I'll try to use that next time I'll do a configure update.

Thanks,
- Tom


> Change-Id: I9cc2c3b48b1dc30c31a7143563c893fd6f426a0a
> ---
>  gdb/configure    | 8 ++++----
>  gdb/configure.ac | 8 ++++----
>  2 files changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/gdb/configure b/gdb/configure
> index 90c1e3d7948b..448167f3c6d9 100755
> --- a/gdb/configure
> +++ b/gdb/configure
> @@ -12138,8 +12138,8 @@ ac_compile='$CXX -c $CXXFLAGS $CPPFLAGS conftest.$ac_ext >&5'
>  ac_link='$CXX -o conftest$ac_exeext $CXXFLAGS $CPPFLAGS $LDFLAGS conftest.$ac_ext $LIBS >&5'
>  ac_compiler_gnu=$ac_cv_cxx_compiler_gnu
>  
> -      save_CFLAGS=$CFLAGS
> -      save_LDFLAGS=$LDFLAGS
> +      save_CFLAGS="$CFLAGS"
> +      save_LDFLAGS="$LDFLAGS"
>        CFLAGS="$CFLAGS $srchigh_pkg_cflags"
>        LDFLAGS="$LDFLAGS $srchigh_pkg_libs"
>        cat confdefs.h - <<_ACEOF >conftest.$ac_ext
> @@ -12163,8 +12163,8 @@ else
>  fi
>  rm -f core conftest.err conftest.$ac_objext \
>      conftest$ac_exeext conftest.$ac_ext
> -      CFLAGS="$SAVE_CFLAGS"
> -      LDFLAGS="$SAVE_LDFLAGS"
> +      CFLAGS="$save_CFLAGS"
> +      LDFLAGS="$save_LDFLAGS"
>        ac_ext=c
>  ac_cpp='$CPP $CPPFLAGS'
>  ac_compile='$CC -c $CFLAGS $CPPFLAGS conftest.$ac_ext >&5'
> diff --git a/gdb/configure.ac b/gdb/configure.ac
> index 29c1c4589c95..ada937316d73 100644
> --- a/gdb/configure.ac
> +++ b/gdb/configure.ac
> @@ -1265,8 +1265,8 @@ either use --disable-source-highlight or dnl
>        # This situation can occur for instance when using a source highlight
>        # library compiled with g++ 7.5.0 while building gdb with g++ 4.8.5.
>        AC_LANG_PUSH(C++)
> -      save_CFLAGS=$CFLAGS
> -      save_LDFLAGS=$LDFLAGS
> +      save_CFLAGS="$CFLAGS"
> +      save_LDFLAGS="$LDFLAGS"
>        CFLAGS="$CFLAGS $srchigh_pkg_cflags"
>        LDFLAGS="$LDFLAGS $srchigh_pkg_libs"
>        AC_LINK_IFELSE(
> @@ -1278,8 +1278,8 @@ either use --disable-source-highlight or dnl
>          [have_usable_source_highlight=yes],
>          [have_usable_source_highlight=no]
>        )
> -      CFLAGS="$SAVE_CFLAGS"
> -      LDFLAGS="$SAVE_LDFLAGS"
> +      CFLAGS="$save_CFLAGS"
> +      LDFLAGS="$save_LDFLAGS"
>        AC_LANG_POP(C++)
>  
>        if test "${have_usable_source_highlight}" = "yes"; then
>