Skip to content

Commit

Permalink
Use ldflags instead of LDFLAGS in newlib.exp
Browse files Browse the repository at this point in the history
This variable was accidentally renamed earlier. It must be
ldflags according to DejaGNU documentation.

Signed-off-by: Claudiu Zissulescu <[email protected]>
  • Loading branch information
claziss authored and github-cygwin committed Aug 21, 2024
1 parent 86d63f0 commit d58915a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions newlib/testsuite/lib/newlib.exp
Original file line number Diff line number Diff line change
Expand Up @@ -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]"
}
}

Expand Down

0 comments on commit d58915a

Please sign in to comment.