[Buildroot] [PATCH 21/23] python-pyhamcrest: new package

Asaf Kahlon asafka7 at gmail.com
Thu Oct 4 19:30:38 UTC 2018


Thomas, Yegor,

On Thu, Oct 4, 2018 at 3:02 PM Thomas Petazzoni
<thomas.petazzoni at bootlin.com> wrote:
>
> Hello,
>
> On Wed, 26 Sep 2018 18:35:03 +0300, Asaf Kahlon wrote:
> > Hamcrest framework for matcher objects.
> >
> > Signed-off-by: Asaf Kahlon <asafka7 at gmail.com>
> > ---
> >  DEVELOPERS                                       |  1 +
> >  package/Config.in                                |  1 +
> >  package/python-pyhamcrest/Config.in              |  8 ++++++++
> >  package/python-pyhamcrest/python-pyhamcrest.hash |  5 +++++
> >  package/python-pyhamcrest/python-pyhamcrest.mk   | 14 ++++++++++++++
> >  5 files changed, 29 insertions(+)
> >  create mode 100644 package/python-pyhamcrest/Config.in
> >  create mode 100644 package/python-pyhamcrest/python-pyhamcrest.hash
> >  create mode 100644 package/python-pyhamcrest/python-pyhamcrest.mk
>
> I've applied to master with Yegor Reviewed-by. However, I have a
> question, see below.
>
> > +config BR2_PACKAGE_PYTHON_PYHAMCREST
> > +     bool "python-pyhamcrest"
> > +     select BR2_PACKAGE_PYTHON_SETUPTOOLS # runtime
>
> How do you know, or how does scanpypi knows when setuptools is needed
> on the target vs. when it's only needed at build time on the build
> machine ?

As far as I know, there's no special treatment for setuptools in
scanpypi. For pyhamcrest, setuptools was added since it was explicitly
mentioned under install_requires.
Generally, not adding setuptools to install_requires is much like not
adding any other package and scanpypi won't catch this. (Yegor,
correct me if I am wrong).
Most of the times, I try to run python and import the package for
sanity check, but (at least from a first glance) I don't think there's
a way to know exactly if setuptools is needed on target although not
specified in install_requires.

>
> Thanks,
>
> Thomas
> --
> Thomas Petazzoni, CTO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com

Regards,
Asaf.


More information about the buildroot mailing list