[Buildroot] [RFC PATCH 3/4] toolchain-wrapper: add linker override option

[Arnout Vandecappelle]

On 24/07/2019 19:35, Joseph Kogut wrote:
> Signed-off-by: Joseph Kogut <joseph.kogut at gmail.com>
> ---
>  toolchain/toolchain-wrapper.c | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/toolchain/toolchain-wrapper.c b/toolchain/toolchain-wrapper.c
> index c73a0cc079..131a239f7f 100644
> --- a/toolchain/toolchain-wrapper.c
> +++ b/toolchain/toolchain-wrapper.c
> @@ -360,6 +360,17 @@ int main(int argc, char **argv)
>  	}
>  #endif /* ARCH || CPU */
>  
> +#if defined(BR_LINKER)
> +	/* Specify our linker if it's not already set */
> +	for (i = 1; i < argc; i++) {
> +		if (!strncmp(argv[i], "-fuse-ld=", strlen("-fuse-ld=")))
> +			break;
> +	}
> +	if (i == argc && !getenv("BR2_NO_LINKER_OVERRIDE")) {

 A while ago we decided that "internal" environment variables should use BR_
prefix, while "user" variables should use BR2_. I'm not sure if we consistently
do that, but I do think that here we should.


 Regards,
 Arnout

> +		*cur++ = "-fuse-ld=" BR_LINKER;
> +	}
> +#endif /* BR_LINKER */
> +
>  	if (parse_source_date_epoch_from_env()) {
>  		*cur++ = _time_;
>  		*cur++ = _date_;
>