* [Buildroot] [PATCH] binutils: add comment with bug report reference
@ 2016-10-30 13:25 Waldemar Brodkorb
2016-10-30 15:43 ` Thomas Petazzoni
2016-11-01 15:18 ` Thomas Petazzoni
0 siblings, 2 replies; 5+ messages in thread
From: Waldemar Brodkorb @ 2016-10-30 13:25 UTC (permalink / raw)
To: buildroot
Add requested bug report
Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
---
package/binutils/Config.in.host | 1 +
1 file changed, 1 insertion(+)
diff --git a/package/binutils/Config.in.host b/package/binutils/Config.in.host
index a7610b4..dce6996 100644
--- a/package/binutils/Config.in.host
+++ b/package/binutils/Config.in.host
@@ -18,6 +18,7 @@ choice
config BR2_BINUTILS_VERSION_2_27_X
bool "binutils 2.27"
# supported but broken on Microblaze
+ # see https://sourceware.org/bugzilla/show_bug.cgi?id=20748
depends on !BR2_microblaze
endchoice
--
2.1.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] binutils: add comment with bug report reference
2016-10-30 13:25 [Buildroot] [PATCH] binutils: add comment with bug report reference Waldemar Brodkorb
@ 2016-10-30 15:43 ` Thomas Petazzoni
2016-10-30 16:18 ` Waldemar Brodkorb
2016-11-01 15:18 ` Thomas Petazzoni
1 sibling, 1 reply; 5+ messages in thread
From: Thomas Petazzoni @ 2016-10-30 15:43 UTC (permalink / raw)
To: buildroot
Hello,
On Sun, 30 Oct 2016 14:25:11 +0100, Waldemar Brodkorb wrote:
> Add requested bug report
>
> Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
> ---
> package/binutils/Config.in.host | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/package/binutils/Config.in.host b/package/binutils/Config.in.host
> index a7610b4..dce6996 100644
> --- a/package/binutils/Config.in.host
> +++ b/package/binutils/Config.in.host
> @@ -18,6 +18,7 @@ choice
> config BR2_BINUTILS_VERSION_2_27_X
> bool "binutils 2.27"
> # supported but broken on Microblaze
> + # see https://sourceware.org/bugzilla/show_bug.cgi?id=20748
> depends on !BR2_microblaze
Your bug report is about binutils 2.27 generating broken binaries for
microblaze/musl, but you disable binutils 2.27 completely for
microblaze, and not just for musl. Why so?
Thanks,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] binutils: add comment with bug report reference
2016-10-30 15:43 ` Thomas Petazzoni
@ 2016-10-30 16:18 ` Waldemar Brodkorb
2016-10-30 17:10 ` Thomas Petazzoni
0 siblings, 1 reply; 5+ messages in thread
From: Waldemar Brodkorb @ 2016-10-30 16:18 UTC (permalink / raw)
To: buildroot
Hi Thomas,
Thomas Petazzoni wrote,
> Hello,
>
> On Sun, 30 Oct 2016 14:25:11 +0100, Waldemar Brodkorb wrote:
> > Add requested bug report
> >
> > Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
> > ---
> > package/binutils/Config.in.host | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/package/binutils/Config.in.host b/package/binutils/Config.in.host
> > index a7610b4..dce6996 100644
> > --- a/package/binutils/Config.in.host
> > +++ b/package/binutils/Config.in.host
> > @@ -18,6 +18,7 @@ choice
> > config BR2_BINUTILS_VERSION_2_27_X
> > bool "binutils 2.27"
> > # supported but broken on Microblaze
> > + # see https://sourceware.org/bugzilla/show_bug.cgi?id=20748
> > depends on !BR2_microblaze
>
> Your bug report is about binutils 2.27 generating broken binaries for
> microblaze/musl, but you disable binutils 2.27 completely for
> microblaze, and not just for musl. Why so?
Because the binutils bug affects musl/uClibc-ng and glibc.
I just used musl to bisect and simplified the bug report.
Furthermore I normally get very good support from the musl devs
in case of toolchain issues.
best regards
Waldemar
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] binutils: add comment with bug report reference
2016-10-30 16:18 ` Waldemar Brodkorb
@ 2016-10-30 17:10 ` Thomas Petazzoni
0 siblings, 0 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2016-10-30 17:10 UTC (permalink / raw)
To: buildroot
Hello,
On Sun, 30 Oct 2016 17:18:09 +0100, Waldemar Brodkorb wrote:
> > > config BR2_BINUTILS_VERSION_2_27_X
> > > bool "binutils 2.27"
> > > # supported but broken on Microblaze
> > > + # see https://sourceware.org/bugzilla/show_bug.cgi?id=20748
> > > depends on !BR2_microblaze
> >
> > Your bug report is about binutils 2.27 generating broken binaries for
> > microblaze/musl, but you disable binutils 2.27 completely for
> > microblaze, and not just for musl. Why so?
>
> Because the binutils bug affects musl/uClibc-ng and glibc.
> I just used musl to bisect and simplified the bug report.
> Furthermore I normally get very good support from the musl devs
> in case of toolchain issues.
ACK, thanks for the explanation. But then perhaps your bug report
should be made more accurate? As it is written, one might understand
than it is a musl-specific problem. Knowing that uclibc-ng and glibc
are also affected is quite important IMO.
Thanks,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] binutils: add comment with bug report reference
2016-10-30 13:25 [Buildroot] [PATCH] binutils: add comment with bug report reference Waldemar Brodkorb
2016-10-30 15:43 ` Thomas Petazzoni
@ 2016-11-01 15:18 ` Thomas Petazzoni
1 sibling, 0 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2016-11-01 15:18 UTC (permalink / raw)
To: buildroot
Hello,
On Sun, 30 Oct 2016 14:25:11 +0100, Waldemar Brodkorb wrote:
> Add requested bug report
>
> Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
> ---
> package/binutils/Config.in.host | 1 +
> 1 file changed, 1 insertion(+)
Applied to master, thanks.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-11-01 15:18 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-10-30 13:25 [Buildroot] [PATCH] binutils: add comment with bug report reference Waldemar Brodkorb
2016-10-30 15:43 ` Thomas Petazzoni
2016-10-30 16:18 ` Waldemar Brodkorb
2016-10-30 17:10 ` Thomas Petazzoni
2016-11-01 15:18 ` Thomas Petazzoni
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox