pkarashchenko commented on a change in pull request #5662:
URL: https://github.com/apache/incubator-nuttx/pull/5662#discussion_r817702007
##########
File path: boards/arm/phy62xx/phy6222/scripts/Make.defs
##########
@@ -53,9 +53,6 @@ NXFLATLDFLAGS1 = -r -Wl,-d -Wl,-warn-common
NXFLATLDFLAGS2 = $(NXFLATLDFLAGS1)
-T$(TOPDIR)/binfmt/libnxflat/gnu-nxflat-pcrel.ld -Wl,-no-check-sections
LDNXFLATFLAGS = -e main -s 2048
-ifneq ($(CROSSDEV),arm-nuttx-elf-)
Review comment:
I see the notes in ReleaseNotes
```
### LD Now Called Through GCC
When building NuttX with GCC toolchains, the linker is now called through
$(CROSSDEV)gcc rather than $(CROSSDEV)ld. This requires all linker options to
be prefixed with `-Wl,` (dash, capital W, lower-case L, comma). All boards in
the NuttX tree have been updated, but any custom out-of-tree boards will need
to apply this change in the relevant parts of their build scripts.
The reason for this change is that GNU Binutils 2.36.x has added new error
checking on command line arguments. This has had the effect that the NuttX
build suddenly broke on host OS distributions that updated to the 2.36.x
release line. Reports included Arch, Manjaro, and Ubuntu. Other host OSes are
likely to be affected as well.
#### Symptoms
Your toolchain and board are affected if you see errors from LD like:
```
arm-none-eabi-ld: Error: unable to disambiguate: -nostartfiles (did you mean
--nostartfiles ?)
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
