From: "Andreas Bießmann" <andreas.devel@googlemail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] tools: imagetool: Remove "INSERT BEFORE" from linker script
Date: Mon, 16 Feb 2015 20:50:46 +0100 [thread overview]
Message-ID: <54E24A16.1010303@googlemail.com> (raw)
In-Reply-To: <20150216165509.GL7086@bill-the-cat>
On 16.02.15 17:55, Tom Rini wrote:
> On Mon, Feb 16, 2015 at 03:05:45PM +0100, Lukasz Majewski wrote:
>
>> Not all linkers are able to parse "INSERT BEFORE" part of the linker script.
>> For example following (rather new) "gold" one:
>>
>> GNU gold (GNU Binutils for Debian 2.22) 1.11
>>
>> produces following error:
>> HOSTLD tools/mkenvimage
>> /usr/bin/ld: error: ./tools/imagetool.lds:23:8: syntax error, unexpected STRING
>> /usr/bin/ld: fatal error: unable to parse script file ./tools/imagetool.lds
>> collect2: error: ld returned 1 exit status
>> make[1]: *** [tools/mkenvimage] Error 1
>> make: *** [tools] Error 2
>>
>> The problem has already been reported, but no evident solution has been
>> proposed:
>> https://bugzilla.redhat.com/show_bug.cgi?id=927573#c5
>>
>> Signed-off-by: Lukasz Majewski <l.majewski@samsung.com>
>
> Guilherme, can you please confirm that things still work as expected for
> you with this patch applied? Thanks!
I vote for 437762 [1] as replacement for this patch.
It is not only the linker script which is not understood by the Darwin
linker. The linker generated lists work just differently on Darwin,
therefore a93648d197df48fa46dd55f925ff70468bd81c71 broke mkimage for me.
Guilherme told the RFC would work [2].
Best regards
Andreas Bie?mann
[1] http://patchwork.ozlabs.org/patch/437762/
[2] http://article.gmane.org/gmane.comp.boot-loaders.u-boot/211224
next prev parent reply other threads:[~2015-02-16 19:50 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-16 14:05 [U-Boot] [PATCH] tools: imagetool: Remove "INSERT BEFORE" from linker script Lukasz Majewski
2015-02-16 16:55 ` Tom Rini
2015-02-16 17:12 ` Tom Rini
2015-02-16 17:47 ` Guilherme Ferreira
2015-02-16 19:50 ` Andreas Bießmann [this message]
2015-02-16 19:57 ` Tom Rini
2015-02-17 8:04 ` Lukasz Majewski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=54E24A16.1010303@googlemail.com \
--to=andreas.devel@googlemail.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox