Joel Stanley <j...@jms.id.au> writes: > Hello Tom, > > On Sat, Dec 10, 2016 at 12:53 AM, Tom Rini <tr...@konsulko.com> wrote: >>> -LDFLAGS_u-boot += $(LDFLAGS_FINAL) >>> +# Avoid Not enough room for program headers on binutils 2.28 onwards. >>> +# Flag was introduced in 2.26 >>> +LDFLAGS_u-boot += $(LDFLAGS_FINAL) \ >>> + $(call ld-ifversion, -ge, 22600000, --no-dynamic-linker) >> >> This breaks on things like: >> $ arm-none-eabi-ld --version >> GNU ld (GNU Tools for ARM Embedded Processors) 2.24.0.20150921 > > The flag will only be added when the version is >= 2.26. Which part of > that version string will break the test? > >> so please use ld-option instead to add this when supported, thanks! > > If you'd just prefer ld-option over ld-ifversion then I can send a > different patch.
Always test for the actual thing you need whenever possible. Version checks are fragile and should be a last resort. Consider what would happen if someone backported that functionality to an older binutils version. Going by version number alone, the flag would not be used and the build would fail. -- Måns Rullgård _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot