[PATCH] newlib: stop clobbering LDFLAGS with non-standard $ldflags
Corinna Vinschen
vinschen@redhat.com
Fri Jan 21 11:25:45 GMT 2022
On Jan 20 23:51, Mike Frysinger wrote:
> It's unclear why this was added originally, but assuming it was needed
> 20 years ago, it shouldn't be explicitly required nowadays. Current
> versions of autotools already take care of exporting LDFLAGS to the
> Makefile as needed (things are actually getting linked). That's why
> the configure diffs show LDFLAGS still here, but shifted to a diff
> place in the output list. A few dirs stop exporting LDFLAGS, but
> that's because they don't do any linking, only compiling, so it's
> correct.
>
> As for the use of $ldflags instead of the standard $LDFLAGS, I can't
> really explain that at all. Just use the right name so users don't
> have to dig into why their setting isn't respected, and then use a
> non-standard name instead. Adjust the testsuite to match.
LGTM, thanks,
Corinna
More information about the Newlib
mailing list