Re: [PATCH 4/6] newlib: drop libtool support

[Corinna Vinschen <vinschen@redhat.com>]

On Feb  7 20:57, Mike Frysinger wrote:
> This was only ever used for i?86-pc-linux-gnu targets, but that's been
> broken for years, and has since been dropped.  So clean this up too.
> 
> This also deletes the funky objectlist logic since it only existed for
> the libtool libraries.  Since it was the only thing left in the small
> Makefile.shared file, we can punt that too.
> ---
> [...]
> diff --git a/newlib/libm/configure.ac b/newlib/libm/configure.ac
> index a03ea3872bd5..7b4add03f3af 100644
> --- a/newlib/libm/configure.ac
> +++ b/newlib/libm/configure.ac
> @@ -20,15 +20,6 @@ AC_NO_EXECUTABLES
>  NEWLIB_CONFIGURE(..)
>  AC_PROG_CPP
>  
> -dnl We have to enable libtool after NEWLIB_CONFIGURE because if we try and
> -dnl add it into NEWLIB_CONFIGURE, executable tests are made before the first
> -dnl line of the macro which fail because appropriate LDFLAGS are not set.
> -_LT_DECL_SED
> -_LT_PROG_ECHO_BACKSLASH
> -if test "${use_libtool}" = "yes"; then
> -LT_INIT([win32-dll])
> -fi
> -
>  AC_TYPE_LONG_DOUBLE
>  AM_CONDITIONAL(HAVE_LONG_DOUBLE, test x"$ac_cv_type_long_double" = x"yes")
>  
> @@ -53,12 +44,8 @@ if test -n "${libm_machine_dir}"; then
>      *) AC_MSG_ERROR([unsupported libm_machine_dir "${libm_machine_dir}"]) ;;
>    esac
>  
> -  LIBM_MACHINE_DIR=machine/${libm_machine_dir}
> -  if test "${use_libtool}" = "yes"; then
> -    LIBM_MACHINE_LIB=${LIBM_MACHINE_DIR}/lib${libm_machine_dir}.${aext}
> -  else
> -    LIBM_MACHINE_LIB=${LIBM_MACHINE_DIR}/lib.${aext}
> -  fi
> +  LIBM_MACHINE_DIR=machine/${libm_machine_dir
                              ^^^^^^^^^^^^^^^^^^^
                              oops?


Corinna