Re: [Qemu-trivial] [PATCH] configure: Use ${config_host_ld} variable

[Michael Tokarev]

05.06.2013 19:02, Ed Maste wrote:
> -  linker_script="-Wl,-T../config-host.ld 
> -Wl,-T,\$(SRC_PATH)/ldscripts/\$(ARCH).ld"
> +  linker_script="-Wl,-T../${config_host_ld} 
> -Wl,-T,\$(SRC_PATH)/ldscripts/\$(ARCH).ld"

Do we really care?  To me it looks like it is better
to just remove the variable (which is nothing but just
an extra indirection) and be done with it.

The variable has been introduced by this commit:

commit 4bf6b55b346d33ff16dd25f75bbea8e21f95dfa6
Author: Juan Quintela <address@hidden>
Date:   Wed Jul 22 22:37:40 2009 +0200

    set SEARCH_PATH for the linker script from output of ld --verbose -v

    Signed-off-by: Juan Quintela <address@hidden>
    Signed-off-by: Anthony Liguori <address@hidden>

which description does not cover introduction of
this variable, and apparently it meant to be something
different really.  The thing is: that commit introduced
that envvar, but at the same time, added several
references to open-coded name (config-host.ld) into
several places (including the one which is being
patched now).

I really don't see a point to have it in a variable at all.

Ditto for $config_host_mak for that matter.

Both are used in makefiles as well, and used there
directly.

Maybe just get rid of that cruft?  If there will be
a need for actual variable, it will be trivial to
introduce it and patch in this single place where
it is referenced... :)

Thanks,

/mjt