[PATCH] Use ldflags instead of LDFLAGS in newlib.exp

Corinna Vinschen vinschen@redhat.com
Wed Aug 21 09:18:24 GMT 2024


On Aug 20 14:11, Yuriy.Kolerov@synopsys.com wrote:
> From: Claudiu Zissulescu <claziss@gmail.com>
> 
> This variable was accidentally renamed earlier. It must be
> ldflags according to DejaGNU documentation.
> 
> Signed-off-by: Claudiu Zissulescu <claziss@gmail.com>
> ---
>  newlib/testsuite/lib/newlib.exp | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/newlib/testsuite/lib/newlib.exp b/newlib/testsuite/lib/newlib.exp
> index 8b43d2e5d..f1df40e52 100644
> --- a/newlib/testsuite/lib/newlib.exp
> +++ b/newlib/testsuite/lib/newlib.exp
> @@ -66,13 +66,13 @@ proc newlib_target_compile { source dest type options } {
>  	    [target_info needs_status_wrapper] != "0" && \
>  	    [info exists gluefile] } {
>  	lappend options "libs=$gluefile"
> -	lappend options "LDFLAGS=$wrap_flags"
> +	lappend options "ldflags=$wrap_flags"
>      }
>  
>      if [string match $host_triplet $target_triplet] then {
>  	if [string match "i686-pc-linux-gnu" $host_triplet] then {
>  	    lappend options "libs=$objdir/crt0.o -lc -lgcc"
> -	    lappend options "LDFLAGS=-nostdlib -L$objdir/.libs [newlib_include_flags]"
> +	    lappend options "ldflags=-nostdlib -L$objdir/.libs [newlib_include_flags]"
>  	}
>      }
>  
> -- 
> 2.34.1

Pushed.

Thanks,
Corinna



More information about the Newlib mailing list