From: "Benoît Thébaudeau" <benoit.thebaudeau@advansee.com>
To: buildroot@busybox.net
Subject: [Buildroot] [autobuild.buildroot.net] Build results for 2014-05-11
Date: Thu, 15 May 2014 14:51:03 +0200 (CEST) [thread overview]
Message-ID: <83985810.253657.1400158263188.JavaMail.zimbra@advansee.com> (raw)
In-Reply-To: <20140515095042.14f7c07a@free-electrons.com>
Dear Thomas Petazzoni,
On Thursday, May 15, 2014 9:50:42 AM, Thomas Petazzoni wrote:
> On Wed, 14 May 2014 23:25:53 +0200, Beno?t Th?baudeau wrote:
>
> > > Right. It just papers over the real problem which is building a target
> > > config
> > > test using the host toolchain, and then running it. The patch at
> > > http://patchwork.ozlabs.org/patch/348765/ is better, I believe.
> >
> > The issue in my case was that the native toolchain used for the
> > configure test implicitly #included a header file, which triggered a
> > conflict of direct inclusion between the native and cross toolchains
> > header files. Passing --sysroot forces the native toolchain to only
> > use the header files from the cross toolchain, fixing this conflict.
> > This directly addresses the issue without any assumption regarding the
> > cross libc.
>
> Right, but using target headers with the native toolchain is wrong, and
> potentially also very fragile.
True.
> > http://patchwork.ozlabs.org/patch/348765/ works too, but it removes a
> > configure test and it relies on the "all libc variants we support have
> > the netinet/tcp.h header" assumption, which might become wrong in the
> > future, which is why I didn't choose this solution.
>
> How likely is that to happen? If it does, we will have very clear
> build failures, so we will notice, no?
For this specific test, yes, probably, because of the missing definitions in
Linux, as Baruch pointed out. But for some of the other configure tests, the
script sometimes silently chooses wrong options because of the header files
mix-up, so --sysroot helps here, but it's dirty, and forcing the GLIBC version
in the input options would not fix the other tests.
The root issue is that this configure script is totally broken when
cross-building, so most of its tests would have to be more or less rewritten.
Best regards,
Beno?t
next prev parent reply other threads:[~2014-05-15 12:51 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-12 6:30 [Buildroot] [autobuild.buildroot.net] Build results for 2014-05-11 Thomas Petazzoni
2014-05-13 7:44 ` Thomas Petazzoni
2014-05-13 7:52 ` Baruch Siach
2014-05-13 8:01 ` Thomas Petazzoni
2014-05-14 20:56 ` Baruch Siach
2014-05-14 21:25 ` Benoît Thébaudeau
2014-05-15 7:50 ` Thomas Petazzoni
2014-05-15 12:51 ` Benoît Thébaudeau [this message]
[not found] ` <CA+sos78G3YxRUCd6pj8naGBSHANEp=1f5FrtuG2oeQpbhNbOuw@mail.gmail.com>
2014-05-14 21:41 ` Baruch Siach
2014-05-14 22:10 ` Benoît Thébaudeau
2014-05-15 3:44 ` Baruch Siach
2014-05-15 12:44 ` Benoît Thébaudeau
2014-05-16 5:20 ` Baruch Siach
2014-05-16 10:41 ` Benoît Thébaudeau
2014-05-13 8:44 ` Peter Korsgaard
2014-05-13 8:57 ` Samuel Martin
2014-05-13 9:21 ` Gustavo Zacarias
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=83985810.253657.1400158263188.JavaMail.zimbra@advansee.com \
--to=benoit.thebaudeau@advansee.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.