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 6/8] python-libxml2: new host package
Date: Fri, 28 Oct 2016 15:32:55 +0200	[thread overview]
Message-ID: <20161028153255.5c0e15d3@free-electrons.com> (raw)
In-Reply-To: <1476321239-13894-6-git-send-email-gustavo@zacarias.com.ar>

Gustavo,

On Wed, 12 Oct 2016 22:13:57 -0300, Gustavo Zacarias wrote:
> Add a new python-libxml2 package to build the libxml2 python bindings.
> 
> This is required for host-itstool which in turn is required to build
> evince.
> 
> Unfortunately since we can't detect host-python/host-python3 presence to
> enable it in the libxml2 package itself and leaving it to automatic
> would be problematic just make a separate package for it like
> mesa3d-headers to handle it deterministically.
> 
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>

We discussed this patch during the Developers meeting, and I believe
the general consensus is that we'd instead prefer to introduce hidden
Config.in knobs for host packages.

This is not only useful for this situation, but also for other
situations.

For this one, I see two possibilities:

 * The python/python3 packages gain some hidden host Config.in options
   BR2_PACKAGE_HOST_PYTHON(3). The host-libxml2 package then looks at
   BR2_PACKAGE_HOST_PYTHON(3) and if enabled, then the python support
   in host-libxml2 is enabled.

   Drawback of this approach is that if we want to do it right, every
   package that uses host-python(3) should start selecting this new
   option.

 * The libxml2 package gains a sub-option
   BR2_PACKAGE_HOST_LIBXML2_PYTHON, which allows to enable python
   support. Of course, later we could add BR2_PACKAGE_HOST_LIBXML2 as
   well, and fix the packages that depend on host-libxml2, but it can
   be done later.

Arnout, Peter, which of those two options do you prefer? Or maybe you
were thinking of different approaches?

Thanks,

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

  reply	other threads:[~2016-10-28 13:32 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-13  1:13 [Buildroot] [PATCH 1/8] gsettings-desktop-schemas: bump to version 3.22.0 Gustavo Zacarias
2016-10-13  1:13 ` [Buildroot] [PATCH 2/8] adwaita-icon-theme: install to staging Gustavo Zacarias
2016-10-16 16:15   ` Thomas Petazzoni
2016-10-17 10:08     ` Gustavo Zacarias
2016-10-17 20:30       ` Arnout Vandecappelle
2016-10-13  1:13 ` [Buildroot] [PATCH 3/8] poppler: add explicit cairo support Gustavo Zacarias
2016-10-15  9:33   ` Peter Korsgaard
2016-10-13  1:13 ` [Buildroot] [PATCH 4/8] poppler: add explicit libglib2 support Gustavo Zacarias
2016-10-15  9:34   ` Peter Korsgaard
2016-10-13  1:13 ` [Buildroot] [PATCH 5/8] poppler: bump to version 0.48.0 Gustavo Zacarias
2016-10-15  9:35   ` Peter Korsgaard
2016-10-13  1:13 ` [Buildroot] [PATCH 6/8] python-libxml2: new host package Gustavo Zacarias
2016-10-28 13:32   ` Thomas Petazzoni [this message]
2016-10-28 13:42     ` Gustavo Zacarias
2016-11-04 23:47       ` Arnout Vandecappelle
2016-11-05 10:29         ` Thomas Petazzoni
2016-10-13  1:13 ` [Buildroot] [PATCH 7/8] itstool: " Gustavo Zacarias
2016-10-13  1:13 ` [Buildroot] [PATCH 8/8] evince: new package Gustavo Zacarias
2016-10-15  9:32 ` [Buildroot] [PATCH 1/8] gsettings-desktop-schemas: bump to version 3.22.0 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=20161028153255.5c0e15d3@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