[oe] [PATCH] python-scons_1.3.0: add missing dependencies
Koen Kooi
koen at dominion.thruhere.net
Tue Jan 25 15:59:25 UTC 2011
Some small nitpicks, see below
Op 25 jan 2011, om 16:51 heeft Víctor Manuel Jáquez Leal het volgende geschreven:
> In order to launch scons in a board it depends on a couple python packages
> which were not managed by the recipe.
>
> This patch adds them.
>
> Signed-off-by: Víctor Manuel Jáquez Leal <vjaquez at igalia.com>
> ---
> recipes/python/python-scons_1.3.0.bb | 8 ++++++++
> 1 files changed, 8 insertions(+), 0 deletions(-)
>
> diff --git a/recipes/python/python-scons_1.3.0.bb b/recipes/python/python-scons_1.3.0.bb
> index 792e3a5..0dc3236 100644
> --- a/recipes/python/python-scons_1.3.0.bb
> +++ b/recipes/python/python-scons_1.3.0.bb
> @@ -1,8 +1,16 @@
> DESCRIPTION = "A Software Construction Tool"
> SECTION = "devel/python"
> PRIORITY = "optional"
> +RDEPENDS_${PN} = "python-shell \
> + python-stringold \
> + python-lang \
> + python-io \
> + python-fcntl \
> + python-pickle \
> + python-crypt"
We generally put variables in the order OE uses them, so RDEPENDS should go next to do_package/FILES/etc. Historically it was on top, but that stopped making sense after switching to RDEPENDS_${PN}
> LICENSE = "GPL"
Could you fix that while you're at it? It would be nice to know if it's GPLv2 or GPLv2+.
Other than that, the patch looks OK to me[1].
regards,
Koen
[1] Sconstructs still make my eyes bleed, but that's not the point
More information about the Openembedded-devel
mailing list