Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/2] uclibc: add a missing function member to siginfo.h
Date: Fri, 10 Jan 2014 15:36:15 +0800	[thread overview]
Message-ID: <20140110153615.34044994@skate> (raw)
In-Reply-To: <52CE6FE5.9070305@imgtec.com>

Dear Vicente Olivert Riera,

On Thu, 9 Jan 2014 09:46:13 +0000, Vicente Olivert Riera wrote:

> Yes, I understand what you mean, but what do you want to do? I mean, if 
> uClibc don't release new version, aren't you going to upgrade external 
> toolchains anymore?

Depends on which external toolchains we're talking about. In the
autobuilders, there are various external toolchains that are in use.
Amongst the uClibc based ones, we have:

 * Toolchains built using Buildroot itself. These are not very
   problematic, as I can rebuild them when a new version of Buildroot
   containing useful backported uClibc patches is released. Though it'd
   be great if I wasn't forced to rebuild these toolchains too often.

 * Toolchains built using Crosstool-NG. Those will *not* contain the
   backported uClibc patches that we have in Buildroot. There is a lot
   less uClibc patch backporting activity going on in Crosstool-NG, if
   any.

 * Toolchains coming from vendors, such as the Analog Devices
   toolchain. There's nothing we can do about them.

Therefore, having Buildroot packages relying on so many backported
uClibc patches is getting more and more problematic.

> I think we should fix this for the Buildroot toolchain, because is "our" 
> toolchain and is "our" problem. If external toolchains don't apply those 
> patches, then that will be their problem. They can upgrade the toolchain 
> packaging uClibc to a git version number, so no releases is needed. Of 
> course I understand that having a release is better because that means 
> that the state is consistent, it shouldn't fail, has been tested, etc., 
> but if that's not happening, again, is not our problem.

I think this is completely the wrong way of seeing things. Buildroot's
policy is that we should deviate as little as possible from upstream.
We don't have the man-power to maintain our own set of crazy patches on
top of this or that component. Whenever someone wanted to post patches
adding features to this or that component, we've always rejected them,
and encouraged the contributor to get them merged in the upstream
component instead.

I think it should be the same with uClibc. If some package doesn't work
with an uClibc release, then we should simply exclude this package
entirely from being built with any uClibc toolchain. And if people are
not happy with that, then they should either engage with the
upstream uClibc to ensure that they do releases at reasonable
intervals, or they should realize that uClibc is a near-dead project,
and therefore that switching to another C library probably makes sense.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

  reply	other threads:[~2014-01-10  7:36 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-08 17:32 [Buildroot] [PATCH 1/2] uclibc: add a missing function member to siginfo.h Vicente Olivert Riera
2014-01-08 17:32 ` [Buildroot] [PATCH 2/2] uclibc: Make __SIGEV_PAD_SIZE to take __WORDSIZE into account Vicente Olivert Riera
2014-10-12  7:46   ` Peter Korsgaard
2014-01-09  2:40 ` [Buildroot] [PATCH 1/2] uclibc: add a missing function member to siginfo.h Thomas Petazzoni
2014-01-09  9:46   ` Vicente Olivert Riera
2014-01-10  7:36     ` Thomas Petazzoni [this message]
2014-01-10 10:10       ` Vicente Olivert Riera
2014-01-28 21:25         ` Thomas Petazzoni
2014-10-11 15:16 ` [Buildroot] [PATCH] rt-tests: enable on MIPS uClibc again Arnout Vandecappelle
2014-10-12  7:47   ` Peter Korsgaard
2014-10-12  7:46 ` [Buildroot] [PATCH 1/2] uclibc: add a missing function member to siginfo.h 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=20140110153615.34044994@skate \
    --to=thomas.petazzoni@free-electrons.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