All of lore.kernel.org
 help / color / mirror / Atom feed
From: Romain Naour <romain.naour@gmail.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCHv2 1/2] package/elf2flt: bump version to fix build with Binutils >= 2.32
Date: Wed, 13 Mar 2019 00:14:55 +0100	[thread overview]
Message-ID: <36c243e2-181e-2bae-add6-e481219fc7e3@gmail.com> (raw)
In-Reply-To: <20190312225644.21b59dea@windsurf>

Hi Thomas,

Le 12/03/2019 ? 22:56, Thomas Petazzoni a ?crit?:
> Hello Romain,
> 
> On Sun, 10 Mar 2019 11:44:27 +0100
> Romain Naour <romain.naour@smile.fr> wrote:
> 
>> From: Romain Naour <romain.naour@gmail.com>
>>
>> Binutils added bfd_stdint.h to bfd.h [1], so elf2flt must create a
>> symlink for this header.
>>
>> Bump the elf2flt version that include the upstream fix [2].
>>
>> Keep our local patch 0001-ld-elf2flt-behave-properly-when-called-with-a-name-d.patch
>> that was not integrated with the recent project activity [3].
>>
>> Fixes:
>> [armv7m-uclibc]
>> https://gitlab.com/kubu93/toolchains-builder/-/jobs/148356410
>>
>> [m68k-coldfire-uclibc]
>> https://gitlab.com/kubu93/toolchains-builder/-/jobs/148356412
>>
>> [1] https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commitdiff;h=2d5d5a8f0a8b5a03454bf168b7fa7024bb1ebbd8
>> [2] https://github.com/uclinux-dev/elf2flt/commit/429521f695c6f4212d809a141bfa4417b733f395
>> [3] https://github.com/uclinux-dev/elf2flt/pull/8
>>
>> Signed-off-by: Romain Naour <romain.naour@gmail.com>
> 
> I've applied both patches. I have pinged on the elf2flt pull request as
> well.

Thanks!

> 
> Regarding binutils, we now support 5 versions, which is not good, we
> should get back to just 3 versions, and move to binutils 2.31 as the
> default. Do you think you could send patches doing that ?

Done.

We postponed the removal of binutils 2.28 due to ARM thumb bug IIRC [1].
binutils 2.30 is the only remaining one with this bug, I updated the comment in
linux/Config.in accordingly.

Now binutils 2.31 and 2.32 are out we can remove binutils 2.28 and 2.29.

[1] https://sourceware.org/bugzilla/show_bug.cgi?id=21458

Best regards,
Romain

> 
> Thanks!
> 
> Thomas
> 

      reply	other threads:[~2019-03-12 23:14 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-03-10 10:44 [Buildroot] [PATCHv2 1/2] package/elf2flt: bump version to fix build with Binutils >= 2.32 Romain Naour
2019-03-10 10:44 ` [Buildroot] [PATCHv2 2/2] package/binutils: add support for Binutils 2.32 Romain Naour
2019-03-12 21:56 ` [Buildroot] [PATCHv2 1/2] package/elf2flt: bump version to fix build with Binutils >= 2.32 Thomas Petazzoni
2019-03-12 23:14   ` Romain Naour [this message]

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=36c243e2-181e-2bae-add6-e481219fc7e3@gmail.com \
    --to=romain.naour@gmail.com \
    --cc=buildroot@busybox.net \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.