[Buildroot] [PATCH 1/2] Add pyyaml package

[Buildroot] [PATCH 1/2] Add pyyaml package

[Geoffrey Ragot]

Signed-off-by: Geoffrey Ragot <geoffrey.ragot@gmail.com>
---
 package/Config.in                      |  1 +
 package/python-pyyaml/Config.in        |  8 ++++++++
 package/python-pyyaml/python-pyyaml.mk | 15 +++++++++++++++
 3 files changed, 24 insertions(+)
 create mode 100644 package/python-pyyaml/Config.in
 create mode 100644 package/python-pyyaml/python-pyyaml.mk

diff --git a/package/Config.in b/package/Config.in
index cfa7512..8bc27a4 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -649,6 +649,7 @@ menu "external python modules"
 	source "package/python-pyusb/Config.in"
 	source "package/python-pyxb/Config.in"
 	source "package/python-pyxml/Config.in"
+	source "package/python-pyyaml/Config.in"
 	source "package/python-pyzmq/Config.in"
 	source "package/python-requests/Config.in"
 	source "package/python-rtslib-fb/Config.in"
diff --git a/package/python-pyyaml/Config.in b/package/python-pyyaml/Config.in
new file mode 100644
index 0000000..a63c445
--- /dev/null
+++ b/package/python-pyyaml/Config.in
@@ -0,0 +1,8 @@
+comment "python-pyyaml require libyaml"
+	depends on !BR2_PACKAGE_LIBYAML
+
+config BR2_PACKAGE_PYTHON_PYYAML
+	bool "python-pyyaml"
+	depends on BR2_PACKAGE_LIBYAML
+	help
+	  The PyYAML package contains binding to the libyaml API.
diff --git a/package/python-pyyaml/python-pyyaml.mk b/package/python-pyyaml/python-pyyaml.mk
new file mode 100644
index 0000000..ef10dac
--- /dev/null
+++ b/package/python-pyyaml/python-pyyaml.mk
@@ -0,0 +1,15 @@
+################################################################################
+#
+# python-pyyaml
+#
+################################################################################
+
+PYTHON_PYYAML_VERSION = 3.11
+PYTHON_PYYAML_SOURCE = PyYAML-$(PYTHON_PYYAML_VERSION).tar.gz
+PYTHON_PYYAML_SITE = https://pypi.python.org/packages/source/P/PyYAML/
+PYTHON_PYYAML_SETUP_TYPE = distutils
+PYTHON_PYYAML_LICENSE = Python software foundation license v2
+PYTHON_PYYAML_LICENSE_FILES = LICENSE
+PYTHON_PYYAML_DEPENDENCIES = libyaml
+
+$(eval $(python-package))
-- 
2.4.6

[Buildroot] [PATCH 2/2] Add pycli package

[Geoffrey Ragot]

Signed-off-by: Geoffrey Ragot <geoffrey.ragot@gmail.com>
---
 package/Config.in                    |  1 +
 package/python-pycli/Config.in       |  4 ++++
 package/python-pycli/python-pycli.mk | 15 +++++++++++++++
 3 files changed, 20 insertions(+)
 create mode 100644 package/python-pycli/Config.in
 create mode 100644 package/python-pycli/python-pycli.mk

diff --git a/package/Config.in b/package/Config.in
index 8bc27a4..06a4326 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -633,6 +633,7 @@ menu "external python modules"
 	source "package/python-protobuf/Config.in"
 	source "package/python-psutil/Config.in"
 	source "package/python-pyasn/Config.in"
+	source "package/python-pycli/Config.in"
 	source "package/python-pycrypto/Config.in"
 	source "package/python-pydal/Config.in"
 	source "package/python-pyftpdlib/Config.in"
diff --git a/package/python-pycli/Config.in b/package/python-pycli/Config.in
new file mode 100644
index 0000000..a4f157e
--- /dev/null
+++ b/package/python-pycli/Config.in
@@ -0,0 +1,4 @@
+config BR2_PACKAGE_PYTHON_PYCLI
+	bool "python-pycli"
+	help
+	  The cli package is a framework for making simple, correct command line applications in Python.
diff --git a/package/python-pycli/python-pycli.mk b/package/python-pycli/python-pycli.mk
new file mode 100644
index 0000000..6857425
--- /dev/null
+++ b/package/python-pycli/python-pycli.mk
@@ -0,0 +1,15 @@
+################################################################################
+#
+# python-pycli
+#
+################################################################################
+
+PYTHON_PYCLI_VERSION = devel
+PYTHON_PYCLI_SOURCE = pyCLI-$(PYTHON_PYCLI_VERSION).tar.gz
+PYTHON_PYCLI_SITE = https://pypi.python.org/packages/source/p/pyCLI/
+PYTHON_PYCLI_SETUP_TYPE = distutils
+PYTHON_PYCLI_LICENSE = Python software foundation license v2
+PYTHON_PYCLI_LICENSE_FILES = LICENSE
+PYTHON_PYCLI_SETUP_TYPE = setuptools
+
+$(eval $(python-package))
-- 
2.4.6

[Buildroot] [PATCH 1/2] Add pyyaml package

[Thomas Petazzoni]

Geoffrey,

The commit title should have been:

	python-pyyaml: new package

(just to follow the Buildroot conventions).

On Sun, 26 Jul 2015 14:12:17 +0200, Geoffrey Ragot wrote:

> diff --git a/package/python-pyyaml/Config.in b/package/python-pyyaml/Config.in
> new file mode 100644
> index 0000000..a63c445
> --- /dev/null
> +++ b/package/python-pyyaml/Config.in
> @@ -0,0 +1,8 @@
> +comment "python-pyyaml require libyaml"
> +	depends on !BR2_PACKAGE_LIBYAML

This comment is not needed...

> +
> +config BR2_PACKAGE_PYTHON_PYYAML
> +	bool "python-pyyaml"
> +	depends on BR2_PACKAGE_LIBYAML

... because 'select' should be used instead of 'depends on' for such
dependencies.

> +	help
> +	  The PyYAML package contains binding to the libyaml API.

We normally include the URL of the upstream project here.

> diff --git a/package/python-pyyaml/python-pyyaml.mk b/package/python-pyyaml/python-pyyaml.mk
> new file mode 100644
> index 0000000..ef10dac
> --- /dev/null
> +++ b/package/python-pyyaml/python-pyyaml.mk
> @@ -0,0 +1,15 @@
> +################################################################################
> +#
> +# python-pyyaml
> +#
> +################################################################################
> +
> +PYTHON_PYYAML_VERSION = 3.11
> +PYTHON_PYYAML_SOURCE = PyYAML-$(PYTHON_PYYAML_VERSION).tar.gz
> +PYTHON_PYYAML_SITE = https://pypi.python.org/packages/source/P/PyYAML/
> +PYTHON_PYYAML_SETUP_TYPE = distutils
> +PYTHON_PYYAML_LICENSE = Python software foundation license v2
> +PYTHON_PYYAML_LICENSE_FILES = LICENSE
> +PYTHON_PYYAML_DEPENDENCIES = libyaml
> +
> +$(eval $(python-package))

The .mk file was OK, however, the .hash file was missing.

I've fixed those issues:

    [Thomas:
      - use 'select' rather than 'depends on' for the yaml library
        dependency
      - add hash file.
      - add upstream URL in Config.in help text
      - rework commit title.]

and applied.

Thanks for your contribution!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH 2/2] Add pycli package

[Thomas Petazzoni]

Dear Geoffrey Ragot,

On Sun, 26 Jul 2015 14:12:18 +0200, Geoffrey Ragot wrote:
> Signed-off-by: Geoffrey Ragot <geoffrey.ragot@gmail.com>
> ---
>  package/Config.in                    |  1 +
>  package/python-pycli/Config.in       |  4 ++++
>  package/python-pycli/python-pycli.mk | 15 +++++++++++++++
>  3 files changed, 20 insertions(+)
>  create mode 100644 package/python-pycli/Config.in
>  create mode 100644 package/python-pycli/python-pycli.mk

I've applied this patch, but had to fix a number of different things
before that. See below.

> diff --git a/package/python-pycli/Config.in b/package/python-pycli/Config.in
> new file mode 100644
> index 0000000..a4f157e
> --- /dev/null
> +++ b/package/python-pycli/Config.in
> @@ -0,0 +1,4 @@
> +config BR2_PACKAGE_PYTHON_PYCLI
> +	bool "python-pycli"
> +	help
> +	  The cli package is a framework for making simple, correct command line applications in Python.

This line is too long. Text should be wrapped so that it fits in 72
columns.

Also, Config.in help text should have the URL of the project homepage.

> diff --git a/package/python-pycli/python-pycli.mk b/package/python-pycli/python-pycli.mk
> new file mode 100644
> index 0000000..6857425
> --- /dev/null
> +++ b/package/python-pycli/python-pycli.mk
> @@ -0,0 +1,15 @@
> +################################################################################
> +#
> +# python-pycli
> +#
> +################################################################################
> +
> +PYTHON_PYCLI_VERSION = devel

This is a strange version, which probably indicates they will change
the tarball contents without changing the version. You missed the hash
file, which would allow to check this, so I've added the hash file.

> +PYTHON_PYCLI_SOURCE = pyCLI-$(PYTHON_PYCLI_VERSION).tar.gz
> +PYTHON_PYCLI_SITE = https://pypi.python.org/packages/source/p/pyCLI/
> +PYTHON_PYCLI_SETUP_TYPE = distutils

First definition of <pkg>_SETUP_TYPE.

> +PYTHON_PYCLI_LICENSE = Python software foundation license v2
> +PYTHON_PYCLI_LICENSE_FILES = LICENSE

There was no file named LICENSE in the source directory. And the
contents of the source tree was a bit fuzzy in terms of license,
mentioning some code having Apache-2.0, some other MIT, but the main
file indicating an ISC-like license. So I opted for that one as the
license.

> +PYTHON_PYCLI_SETUP_TYPE = setuptools

And second definition of <pkg>_SETUP_TYPE. So I kept only this one.

Also, as I've said, I've added a hash file as well.

See
http://git.buildroot.net/buildroot/commit/?id=1ec19e5e47486d2ba61564b37f1c8b6e62ec6d28
for the final commit.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com