Re: [PATCH] config-host.mak: remove unused compiler-related lines

[Thomas Huth]

On 06/07/2021 09.29, Paolo Bonzini wrote:
> Most of the build is not done via Makefiles, therefore the toolchain
> variables are mostly unused.  They are still used by tests/tcg
> and pc-bios/roms, but most of them are not needed there.
>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>   configure   | 10 ----------
>   meson.build |  1 -
>   2 files changed, 11 deletions(-)
>
> diff --git a/configure b/configure
> index 0e487ae091..650d9c0735 100755
> --- a/configure
> +++ b/configure
> @@ -521,9 +521,6 @@ query_pkg_config() {
>   pkg_config=query_pkg_config
>   sdl2_config="${SDL2_CONFIG-${cross_prefix}sdl2-config}"
>   
> -# If the user hasn't specified ARFLAGS, default to 'rv', just as make does.
> -ARFLAGS="${ARFLAGS-rv}"
> -
>   # default flags for all hosts
>   # We use -fwrapv to tell the compiler that we require a C dialect where
>   # left shift of signed integers is well defined and has the expected
> @@ -4917,19 +4914,12 @@ echo "HOST_CC=$host_cc" >> $config_host_mak
>   if $iasl -h > /dev/null 2>&1; then
>     echo "CONFIG_IASL=$iasl" >> $config_host_mak
>   fi
> -echo "CXX=$cxx" >> $config_host_mak
> -echo "OBJCC=$objcc" >> $config_host_mak
>   echo "AR=$ar" >> $config_host_mak
> -echo "ARFLAGS=$ARFLAGS" >> $config_host_mak
>   echo "AS=$as" >> $config_host_mak
>   echo "CCAS=$ccas" >> $config_host_mak
>   echo "CPP=$cpp" >> $config_host_mak
>   echo "OBJCOPY=$objcopy" >> $config_host_mak
>   echo "LD=$ld" >> $config_host_mak
> -echo "RANLIB=$ranlib" >> $config_host_mak
> -echo "NM=$nm" >> $config_host_mak
> -echo "PKG_CONFIG=$pkg_config_exe" >> $config_host_mak
> -echo "WINDRES=$windres" >> $config_host_mak

I wasn't quite sure whether we'd still need some of those e.g. for compiling 
capstone or dtc, but looking at meson.build again, I think you're right, 
we're not using "make" to build those anymore, right?

In that case:

Reviewed-by: Thomas Huth <thuth@redhat.com>