All of lore.kernel.org
 help / color / mirror / Atom feed
From: Luca Ceresoli <luca@lucaceresoli.net>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] audiofile: does not build with static-only
Date: Sun, 25 May 2014 23:00:00 +0200	[thread overview]
Message-ID: <538259D0.4090008@lucaceresoli.net> (raw)
In-Reply-To: <5381278F.9040209@zacarias.com.ar>

Hi Gustavo, Thomas,

Gustavo Zacarias wrote:
> On 05/24/2014 01:45 PM, Thomas Petazzoni wrote:
>
>> No, I disagree. The real problem is the fact that we pass --static
>> instead of -static when building statically, so libtool turns -lstdc++
>> into a full path to libstdc++.so instead of libstdc++.a.
>>
>> The real fix is to understand why we switched from -static to --static
>> a few years ago: there is a commit from Andy Kennedy making this
>> change, but the commit log is quite fuzzy about why this is actually
>> needed.
>
> Hi.
> As we discussed with Thomas on IRC i've been digging and testing a bit
> and promised to write about my assorted notes for the static dilemma:

Ah, I see, I hadn't followed that discussion and was not aware of such
work you're doing. Sorry for the noise and thanks for the detailed
clarification.

I understand the solution won't land in 2014.05.
I wonder what's the best thing to do in cases like this to make users
aware what there's a known bug in a release.

Should audiofile depend on BROKEN if BR2_PREFER_STATIC_LIB? This way it
would disappear from both the user's sight and the autobuilders, but
still visible via menuconfig search.

Or should we just add a comment that depends on audiofile and
BR2_PREFER_STATIC_LIB?

I think the former (o the sum of the two) is preferred. It would stop
polluting the autobuilders with known bugs and track the issue via
`git grep BROKEN`.

Helping in fixing this bug is out of reach for me, but I can at least
send a patch for BROKEN if it's the proper way to go.

-- 
Luca

  parent reply	other threads:[~2014-05-25 21:00 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-24 14:24 [Buildroot] [PATCH] audiofile: does not build with static-only Luca Ceresoli
2014-05-24 16:45 ` Thomas Petazzoni
2014-05-24 23:13   ` Gustavo Zacarias
2014-05-25 20:39     ` Peter Korsgaard
2014-05-25 21:00     ` Luca Ceresoli [this message]
2014-05-25 21:29       ` Gustavo Zacarias
2014-05-28 16:15       ` Arnout Vandecappelle
2014-05-29  7:54         ` Thomas Petazzoni
2014-05-26  9:34     ` 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=538259D0.4090008@lucaceresoli.net \
    --to=luca@lucaceresoli.net \
    --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.