From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2 3/3] package/piglit: new package
Date: Tue, 18 Sep 2018 09:51:08 +0200 [thread overview]
Message-ID: <20180918095108.1a128bcf@windsurf> (raw)
In-Reply-To: <84ded507-27f9-9e33-a116-becc8dda57f0@smile.fr>
Hello,
On Mon, 17 Sep 2018 22:10:02 +0200, Romain Naour wrote:
> > This is surprising, what does it do on the host during the build
> > process that requires those tools ? Or is it that piglit generates a
> > report on the target, and then provides a bunch of host tool to process
> > such reports ?
>
> Reports can be processed on the target directly.
> I believe it's for the same reason that we need host-numpy.
What "same reason" ?
> I think the build system is checking the host-python instead of python for the
> target.
>
> I'm not sure how to fix this...
So you mean that all those Python packages are not needed on the host,
but because the build system is bogus and checks their presence on the
host, we have to have them as dependencies ?
So indeed, there's a macro in cmake/Modules/PythonModule.cmake that is
used to check for Mako, Numpy and Six, and it works by running the host
Python. I imagine they are in fact not needed on the host, and this
macro needs to be adjusted. However, I'm not sure what's the right way
to test if a *target* Python module is available.
Maybe Yegor can help on this ? :-)
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
prev parent reply other threads:[~2018-09-18 7:51 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-11 16:21 [Buildroot] [PATCH v2 1/3] package/python-numpy: add host variant for piglit Romain Naour
2018-02-11 16:21 ` [Buildroot] [PATCH v2 2/3] package/waffle: new package Romain Naour
2018-03-30 13:54 ` Valentin Korenblit
2018-09-16 21:30 ` Thomas Petazzoni
2018-09-17 19:59 ` Romain Naour
2018-09-18 7:31 ` Thomas Petazzoni
2018-02-11 16:21 ` [Buildroot] [PATCH v2 3/3] package/piglit: " Romain Naour
2018-03-30 14:01 ` Valentin Korenblit
2018-04-01 17:07 ` Romain Naour
2018-09-16 21:38 ` Thomas Petazzoni
2018-09-17 20:10 ` Romain Naour
2018-09-18 7:51 ` 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=20180918095108.1a128bcf@windsurf \
--to=thomas.petazzoni@bootlin.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