* [Buildroot] [PATCH v2 1/2] python-markupsafe: allow build as host package
@ 2015-01-26 17:25 Erico Nunes
2015-01-26 17:25 ` [Buildroot] [PATCH v2 2/2] python-jinja2: allow build as host-package Erico Nunes
0 siblings, 1 reply; 6+ messages in thread
From: Erico Nunes @ 2015-01-26 17:25 UTC (permalink / raw)
To: buildroot
python-markupsafe is a dependency for python-jinja2, which can be used
during build time as a host tool to generate code based on templates.
This package is being added host package support mostly to allow
host-python-jinja2 to build.
This has been tested to build and work correctly as a host tool as well.
Signed-off-by: Erico Nunes <erico.nunes@datacom.ind.br>
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
package/python-markupsafe/python-markupsafe.mk | 1 +
1 file changed, 1 insertion(+)
diff --git a/package/python-markupsafe/python-markupsafe.mk b/package/python-markupsafe/python-markupsafe.mk
index 9c7eb14..c815728 100644
--- a/package/python-markupsafe/python-markupsafe.mk
+++ b/package/python-markupsafe/python-markupsafe.mk
@@ -12,3 +12,4 @@ PYTHON_MARKUPSAFE_LICENSE = BSD-3c
PYTHON_MARKUPSAFE_LICENSE_FILES = LICENSE
$(eval $(python-package))
+$(eval $(host-python-package))
--
1.9.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH v2 2/2] python-jinja2: allow build as host-package
2015-01-26 17:25 [Buildroot] [PATCH v2 1/2] python-markupsafe: allow build as host package Erico Nunes
@ 2015-01-26 17:25 ` Erico Nunes
2015-01-27 21:07 ` Arnout Vandecappelle
0 siblings, 1 reply; 6+ messages in thread
From: Erico Nunes @ 2015-01-26 17:25 UTC (permalink / raw)
To: buildroot
Jinja2 is a general purpose templating language for python and can be
required during build time for python scripts that generate code.
While currently there are no in-tree Buildroot packages which depend on
host-python-jinja2, I'm currently having to deal with a proprietary
software build system that requires it.
I have tested that it builds and works correctly as a host python
package for a Buildroot host-python and someone else might require it,
so I'm proposing its inclusion.
python-jinja2 for target builds fine without python-markupsafe as a
build dependency, but when building host-python-jinja2 without
host-python-markupsafe installed, the python package manager tries to
download and install it.
To avoid that and install host-python-markupsafe properly through
Buildroot, the host build dependency is needed. Host support for
python-markupsafe was submitted in a previous patch from this patchset.
Signed-off-by: Erico Nunes <erico.nunes@datacom.ind.br>
---
package/python-jinja2/python-jinja2.mk | 2 ++
1 file changed, 2 insertions(+)
diff --git a/package/python-jinja2/python-jinja2.mk b/package/python-jinja2/python-jinja2.mk
index 49625e3..2105f3a 100644
--- a/package/python-jinja2/python-jinja2.mk
+++ b/package/python-jinja2/python-jinja2.mk
@@ -10,5 +10,7 @@ PYTHON_JINJA2_SITE = http://pypi.python.org/packages/source/J/Jinja2
PYTHON_JINJA2_SETUP_TYPE = setuptools
PYTHON_JINJA2_LICENSE = BSD-3c
PYTHON_JINJA2_LICENSE_FILES = LICENSE
+HOST_PYTHON_JINJA2_DEPENDENCIES = host-python-markupsafe
$(eval $(python-package))
+$(eval $(host-python-package))
--
1.9.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH v2 2/2] python-jinja2: allow build as host-package
2015-01-26 17:25 ` [Buildroot] [PATCH v2 2/2] python-jinja2: allow build as host-package Erico Nunes
@ 2015-01-27 21:07 ` Arnout Vandecappelle
2015-01-27 21:14 ` Thomas Petazzoni
0 siblings, 1 reply; 6+ messages in thread
From: Arnout Vandecappelle @ 2015-01-27 21:07 UTC (permalink / raw)
To: buildroot
On 26/01/15 18:25, Erico Nunes wrote:
> Jinja2 is a general purpose templating language for python and can be
> required during build time for python scripts that generate code.
>
> While currently there are no in-tree Buildroot packages which depend on
> host-python-jinja2, I'm currently having to deal with a proprietary
> software build system that requires it.
Forgot to mention in my last review: in that case, you should also add a
Config.in.host for it (and update package/Config.in.host).
> I have tested that it builds and works correctly as a host python
> package for a Buildroot host-python and someone else might require it,
> so I'm proposing its inclusion.
>
> python-jinja2 for target builds fine without python-markupsafe as a
> build dependency, but when building host-python-jinja2 without
> host-python-markupsafe installed, the python package manager tries to
> download and install it.
Good explanation! :-)
However, this should probably be added as a comment in the .mk file as well -
perhaps a bit shorter, e.g.:
# In host build, setup.py tries to download markupsafe if it is not installed
Regards,
Arnout
> To avoid that and install host-python-markupsafe properly through
> Buildroot, the host build dependency is needed. Host support for
> python-markupsafe was submitted in a previous patch from this patchset.
>
> Signed-off-by: Erico Nunes <erico.nunes@datacom.ind.br>
> ---
> package/python-jinja2/python-jinja2.mk | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/package/python-jinja2/python-jinja2.mk b/package/python-jinja2/python-jinja2.mk
> index 49625e3..2105f3a 100644
> --- a/package/python-jinja2/python-jinja2.mk
> +++ b/package/python-jinja2/python-jinja2.mk
> @@ -10,5 +10,7 @@ PYTHON_JINJA2_SITE = http://pypi.python.org/packages/source/J/Jinja2
> PYTHON_JINJA2_SETUP_TYPE = setuptools
> PYTHON_JINJA2_LICENSE = BSD-3c
> PYTHON_JINJA2_LICENSE_FILES = LICENSE
> +HOST_PYTHON_JINJA2_DEPENDENCIES = host-python-markupsafe
>
> $(eval $(python-package))
> +$(eval $(host-python-package))
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH v2 2/2] python-jinja2: allow build as host-package
2015-01-27 21:07 ` Arnout Vandecappelle
@ 2015-01-27 21:14 ` Thomas Petazzoni
2015-01-27 21:19 ` Arnout Vandecappelle
0 siblings, 1 reply; 6+ messages in thread
From: Thomas Petazzoni @ 2015-01-27 21:14 UTC (permalink / raw)
To: buildroot
Dear Arnout Vandecappelle,
On Tue, 27 Jan 2015 22:07:19 +0100, Arnout Vandecappelle wrote:
> > While currently there are no in-tree Buildroot packages which depend on
> > host-python-jinja2, I'm currently having to deal with a proprietary
> > software build system that requires it.
>
> Forgot to mention in my last review: in that case, you should also add a
> Config.in.host for it (and update package/Config.in.host).
Why ? If it's merely used as a build dependency of another package, I
don't see why we should have a menuconfig entry for it. Indeed, having
a host-python-ninja2 package with no users in-tree is weird, but I
don't think cluttering the "Host utilities" menu with such packages is
the appropriate solution here.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH v2 2/2] python-jinja2: allow build as host-package
2015-01-27 21:14 ` Thomas Petazzoni
@ 2015-01-27 21:19 ` Arnout Vandecappelle
2015-01-28 14:56 ` DATACOM - Erico Nunes
0 siblings, 1 reply; 6+ messages in thread
From: Arnout Vandecappelle @ 2015-01-27 21:19 UTC (permalink / raw)
To: buildroot
On 27/01/15 22:14, Thomas Petazzoni wrote:
> Dear Arnout Vandecappelle,
>
> On Tue, 27 Jan 2015 22:07:19 +0100, Arnout Vandecappelle wrote:
>
>>> While currently there are no in-tree Buildroot packages which depend on
>>> host-python-jinja2, I'm currently having to deal with a proprietary
>>> software build system that requires it.
>>
>> Forgot to mention in my last review: in that case, you should also add a
>> Config.in.host for it (and update package/Config.in.host).
>
> Why ? If it's merely used as a build dependency of another package, I
> don't see why we should have a menuconfig entry for it. Indeed, having
> a host-python-ninja2 package with no users in-tree is weird, but I
> don't think cluttering the "Host utilities" menu with such packages is
> the appropriate solution here.
Hm I guess you're right. So Erico, forget I mentioned it.
Regards,
Arnout
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH v2 2/2] python-jinja2: allow build as host-package
2015-01-27 21:19 ` Arnout Vandecappelle
@ 2015-01-28 14:56 ` DATACOM - Erico Nunes
0 siblings, 0 replies; 6+ messages in thread
From: DATACOM - Erico Nunes @ 2015-01-28 14:56 UTC (permalink / raw)
To: buildroot
Hi Thomas and Arnout,
On 01/27/2015 07:19 PM, Arnout Vandecappelle wrote:
> On 27/01/15 22:14, Thomas Petazzoni wrote:
>> Dear Arnout Vandecappelle,
>>
>> On Tue, 27 Jan 2015 22:07:19 +0100, Arnout Vandecappelle wrote:
>>
>>>> While currently there are no in-tree Buildroot packages which depend on
>>>> host-python-jinja2, I'm currently having to deal with a proprietary
>>>> software build system that requires it.
>>>
>>> Forgot to mention in my last review: in that case, you should also add a
>>> Config.in.host for it (and update package/Config.in.host).
>>
>> Why ? If it's merely used as a build dependency of another package, I
>> don't see why we should have a menuconfig entry for it. Indeed, having
>> a host-python-ninja2 package with no users in-tree is weird, but I
>> don't think cluttering the "Host utilities" menu with such packages is
>> the appropriate solution here.
>
> Hm I guess you're right. So Erico, forget I mentioned it.
>
> Regards,
> Arnout
>
I'll just add the comment in the .mk file as suggested by Arnout in the
other reply and resubmit then.
Thank you for your replies.
Erico
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-01-28 14:56 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-26 17:25 [Buildroot] [PATCH v2 1/2] python-markupsafe: allow build as host package Erico Nunes
2015-01-26 17:25 ` [Buildroot] [PATCH v2 2/2] python-jinja2: allow build as host-package Erico Nunes
2015-01-27 21:07 ` Arnout Vandecappelle
2015-01-27 21:14 ` Thomas Petazzoni
2015-01-27 21:19 ` Arnout Vandecappelle
2015-01-28 14:56 ` DATACOM - Erico Nunes
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.