All of lore.kernel.org
 help / color / mirror / Atom feed
From: Vicenç <vicencb@gmail.com>
To: buildroot@busybox.net
Subject: [Buildroot] argp-standalone broken when optimization is enabled
Date: Thu, 14 Jan 2016 12:54:40 +0100	[thread overview]
Message-ID: <CAAMcf8CKZ+ZFbTgTgdkWGMSiV4Xitnb-PpzsZeOSs800O9ieQA@mail.gmail.com> (raw)
In-Reply-To: <20160114115735.079123f4@free-electrons.com>

On Thu, Jan 14, 2016 at 11:57 AM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Vicen?,
>
> Please keep the Buildroot list in Cc, thanks!
>
> On Thu, 14 Jan 2016 11:26:45 +0100, Vicen? wrote:
>> On Thu, Jan 14, 2016 at 10:19 AM, Thomas Petazzoni
>> <thomas.petazzoni@free-electrons.com> wrote:
>> > Vicen?,
>> >
>> > Thanks for your report.
>> >
>> > On Wed, 13 Jan 2016 18:15:37 +0100, Vicen? wrote:
>> >> Hello,
>> >> could this patch be added to buildroot to fix the issue:
>> >>
>> >> https://dev.openwrt.org/browser/trunk/package/libs/argp-standalone/patches/002-no_optimize.patch
>> >
>> > Could you describe more precisely what doesn't work ? The OpenWRT
>> > patch unfortunately doesn't give any details about what it is fixing.
>>
>> Hello Thomas,
>> In the OpenWRT page there is a link to the bug fixed by this patch:
>> https://dev.openwrt.org/ticket/20460
>> The configuration that fails in my case is when this buildroot options
>> are enabled:
>> BR2_OPTIMIZE_3
>> BR2_TOOLCHAIN_EXTERNAL_GCC_5
>> BR2_TOOLCHAIN_EXTERNAL_CUSTOM_MUSL
>> BR2_PACKAGE_LIBV4L_UTILS
>>
>> The error reported is the same as in OpenWRT.
>> argp is provided with the glibc, but the standalone version is
>> automatically selected when building some packages with musl.
>> So, libc != glibc plus some other specific package triggers the issue.
>
> Thanks for the details. Then I think
> http://review.gluster.org/#/c/6034/ is a better solution than the one
> taken by OpenWRT. Can you try this change ?

Hello Thomas,
I can confirm that
http://review.gluster.org/changes/6034/revisions/b2dfa011a3fdcb7d22764d143517d0fbd1c2a201/patch?zip
works fine.
It just needs to tweak the file paths.
And it is a proper solution, indeed better than just disabling optimizations.

Thanks!
  Vicente.

>
> It originates from https://bugzilla.redhat.com/show_bug.cgi?id=1201898.
>
> Thanks,
>
> Thomas
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com

  reply	other threads:[~2016-01-14 11:54 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-13 17:15 [Buildroot] argp-standalone broken when optimization is enabled Vicenç
2016-01-14  9:19 ` Thomas Petazzoni
2016-01-14 10:49   ` Vicenç
     [not found]   ` <CAAMcf8AcZzNpGrvfOaGO-EBPZHfyzojNsKusZr2RuEfC9xE+iw@mail.gmail.com>
2016-01-14 10:57     ` Thomas Petazzoni
2016-01-14 11:54       ` Vicenç [this message]
2016-01-14 13:20         ` Thomas Petazzoni
2016-01-14 14:31           ` [Buildroot] [PATCH] package/argp-standalone: Fix build with c99 compilers Vicente Bergas
2016-01-14 19:36             ` Peter Korsgaard

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=CAAMcf8CKZ+ZFbTgTgdkWGMSiV4Xitnb-PpzsZeOSs800O9ieQA@mail.gmail.com \
    --to=vicencb@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.