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] problem adding new package libcapn
Date: Wed, 17 Feb 2016 09:36:17 +0100	[thread overview]
Message-ID: <20160217093617.38ad4359@free-electrons.com> (raw)
In-Reply-To: <56C3BEBE.5040901@mind.be>

Hello,

On Wed, 17 Feb 2016 01:28:46 +0100, Arnout Vandecappelle wrote:

> > I have the impression that this 'git submodule' thing is used more and
> > more, and that Buildroot not having support to automatically fetch
> > submodules is causing troubles to a number of people.
> 
>  I think what we really see more and more is that packages are distributed
> straight from git instead of through release tarballs. Tarballs often have the
> submodules handled properly.

Right. And the fact that Github generated tarballs don't have the
contents of the sub-modules.

> > Not later than today, a colleague of mine also asked me about this,
> > since he is packaging something that uses submodules.
> > 
> > But very often, submodules are used to "bundle" libraries that should
> > rather be packaged as separate packages.
> 
>  But in that case there is often a tight dependency between them, which makes
> unbundling tough and usually unwanted by upstream. WebEngine is the perfect example.

It depends on the cases. Apparently for this libcapn case, Johan says
that the jansson version being bundled is the same as the one packaged
in Buildroot.

>  Actually, we are doing a kind of bundling as well (or at least, having tight
> dependencies) by fixating a specific version of each package, and worse yet by
> patching them.

I don't follow you here. We are clearly not doing any bundling: if two
packages needs libfoo, then only one copy of libfoo is going to be
installed.

If however those two packages start to use bundled version of libfoo,
then we are possibly going to have two copies of it in the root
filesystem. Or worst, only one copy, with one overwriting the other.
And in either case, it means that security issues are not handled, etc.

Best regards,

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

      reply	other threads:[~2016-02-17  8:36 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-14 23:01 [Buildroot] problem adding new package libcapn Johan Sagaert
2016-02-14 23:26 ` Johan Sagaert
2016-02-16 21:39 ` Arnout Vandecappelle
2016-02-16 21:45   ` Thomas Petazzoni
2016-02-17  0:28     ` Arnout Vandecappelle
2016-02-17  8:36       ` Thomas Petazzoni [this message]

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=20160217093617.38ad4359@free-electrons.com \
    --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