Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Romain Naour <romain.naour@gmail.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] libcap-ng: bump to version 0.7.7
Date: Sat, 5 Mar 2016 12:35:43 +0100	[thread overview]
Message-ID: <56DAC48F.8050409@gmail.com> (raw)
In-Reply-To: <CANu7NGu5qgahQaStQnrjeWouD_+9j9HmVgcLwFBuqAngod2qqQ@mail.gmail.com>

Hello Sergio,

Le 05/03/2016 00:48, Sergio Prado a ?crit :
> Hi Romain,
> 
> Thanks for reviewing the patch.

You're welcome !

> 
> 2016-03-04 20:13 GMT-03:00 Romain Naour <romain.naour@gmail.com
> <mailto:romain.naour@gmail.com>>:
>>
>> Hi Sergio,
>>
>> Le 05/03/2016 00:00, Sergio Prado a ?crit :
>> > Also disable optimization when compiling for nios2 to prevent a compiler
>> > bug.
>>
>> The bump and the nios2 fix must be two separate patches.
> 
> Right.
> 
>> > Fixes:
>> > http://autobuild.buildroot.net/results/3afc4c006a74fd7512dace90b5f884bf7be4cc58/
>> > http://autobuild.buildroot.net/results/2c53f611a705506ca86fedb0f1129df1d315632c/
>> > http://autobuild.buildroot.net/results/c3d9477688b55987591b5b5efed5ee65499aa0e6/
>> > http://autobuild.buildroot.net/results/052ff24f3985b652c11ac390f342e9ad3d862431/
>>
>> Also the compiler bug is due to an issue with gcc <= 5.2. All theses build
>> issues are trigged by br-nios2-full-2015.11-rc1-71-g90d1299.tar.bz2 toolchain
>> which is based on gcc 4.9.
>>
>> I propose to add an exception in the autobuilder script:
>> http://patchwork.ozlabs.org/patch/582313/
>>
>> Also see:
>> http://lists.busybox.net/pipermail/buildroot/2015-December/146308.html
> 
> Adding an exception in the autobuilder will not fix the problem in production,
> right? The user that wants to use this package with NIOS2 and GCC <= 5.2 will
> still catch this problem. Working around the problem disabling GCC optimizations
> fixes in all cases. What do you think?

I think we need to backport the patch [1] to gcc 4.9.3 but we still need to add
an exception in the autobuilder until the nios2 toolchain is rebuild.

Can you test this patch please:
http://patchwork.ozlabs.org/patch/592357/

Since other version of gcc are not available for nios2, we don't need to
backport this patch.

Thanks for looking at nios2 build failure.

Best regards,
Romain

[1]
https://gcc.gnu.org/git/?p=gcc.git;a=commitdiff;h=3e326557585ffde323867375539f04643fcad29a

> 
>>
>> Best regards,
>> Romain
>>

  reply	other threads:[~2016-03-05 11:35 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-04 23:00 [Buildroot] [PATCH] libcap-ng: bump to version 0.7.7 Sergio Prado
2016-03-04 23:13 ` Romain Naour
2016-03-04 23:48   ` Sergio Prado
2016-03-05 11:35     ` Romain Naour [this message]
2016-03-05 18:05 ` Thomas Petazzoni

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=56DAC48F.8050409@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox