[Buildroot] [PATCH v3 3/4] python-cheetah: add host-package support

gwenhael.goavec gwenj at trabucayre.com
Thu Mar 5 08:40:49 UTC 2015


Dear Thomas Petazzoni
On Wed, 4 Mar 2015 22:49:40 +0100
Thomas Petazzoni <thomas.petazzoni at free-electrons.com> wrote:

> Dear Gwenhael Goavec-Merou,
> 
> On Wed,  4 Mar 2015 17:00:10 +0100, Gwenhael Goavec-Merou wrote:
> 
> > diff --git a/package/python-cheetah/python-cheetah.mk b/package/python-cheetah/python-cheetah.mk
> > index 08076b5..faf2436 100644
> > --- a/package/python-cheetah/python-cheetah.mk
> > +++ b/package/python-cheetah/python-cheetah.mk
> > @@ -9,5 +9,7 @@ PYTHON_CHEETAH_SOURCE = Cheetah-$(PYTHON_CHEETAH_VERSION).tar.gz
> >  PYTHON_CHEETAH_SITE = http://pypi.python.org/packages/source/C/Cheetah
> >  PYTHON_CHEETAH_LICENSE = MIT
> >  PYTHON_CHEETAH_SETUP_TYPE = setuptools
> > +PYTHON_CHEETAH_DEPENDENCIES += host-python-markdown
> 
> Are you sure you need to add host-python-markdown as a dependency to
> the target python-cheetah package ? I.e, I'd like to understand if what
> you need is:
> 
> PYTHON_CHEETAH_DEPENDENCIES = host-python-markdown
> 
>  (i.e adds host-python-markdown to the dependencies of both
>  python-cheetah and host-python-cheetah)
> 
> *OR*
> 
> PYTHON_CHEETAH_DEPENDENCIES = python-markdown
> 
>  (i.e adds python-markdown to the dependencies of python-cheetah, and
>  host-python-markdown to the dependencies of host-python-cheetah)
> 
> *OR*
> 
> HOST_PYTHON_CHEETAH_DEPENDENCIES = host-python-markdown
> 
>  (i.e adds host-python-markdown to the dependencies of
>  host-python-cheetah)
> 
I'm wrong, it's true, host-python-markdown is only needed for
host-python-cheetah. I think the best solution is the second option, ie.
PYTHON_CHEETAH_DEPENDENCIES = python-markdown to be sure to have the correct
dependency in all case (although python-cheetah as not really needed to
python-markdown to be compiles).
>
> Also, use '=' and not '+=' in such situations.
> 
> Thanks!
> 
> Thomas
> -- 
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com

Regards
Gwen


More information about the buildroot mailing list