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] qt5webkit: fix building in Python 3 environment
Date: Fri, 14 Oct 2016 15:40:13 +0200	[thread overview]
Message-ID: <20161014154013.231da448@free-electrons.com> (raw)
In-Reply-To: <1476451574-31822-1-git-send-email-yegorslists@googlemail.com>

Hello,

On Fri, 14 Oct 2016 15:26:14 +0200, yegorslists at googlemail.com wrote:
> From: Yegor Yefremov <yegorslists@googlemail.com>
> 
> qt5webkit requires Python for building stage and can be built with both
> Python 2 and 3. So remove the workaround with linking python binary and
> use Python dependency according to the selected Buildroot's Python
> version.
> 
> Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>

This is pretty much reverting commit
ac16793eaaabfced0312420759e3a66cdaa1ea8e, which says that Python 3 on
the host doesn't work to build qt5webkit.

So if we apply your patch, I believe we will fall again into the
autobuilder issues that commit ac16793eaaabfced0312420759e3a66cdaa1ea8e
was fixing.

Can you explain why your patch is needed, and what now makes qt5webkit
build fine with Python 3 on the host ?

Thanks!

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

  reply	other threads:[~2016-10-14 13:40 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-14 13:26 [Buildroot] [PATCH] qt5webkit: fix building in Python 3 environment yegorslists at googlemail.com
2016-10-14 13:40 ` Thomas Petazzoni [this message]
2016-10-14 13:59   ` Yegor Yefremov
2016-10-14 14:12     ` Arnout Vandecappelle
2016-10-17 10:29       ` Yegor Yefremov
2016-10-17 12:15         ` 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=20161014154013.231da448@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