diff mbox

[2/9] package/python-traitlets: New package

Message ID 20170425191423.13294-3-andrew.smirnov@gmail.com
State Superseded
Headers show

Commit Message

Andrey Smirnov April 25, 2017, 7:14 p.m. UTC
Add 'traitlets'[1] package to Buildroot. Needed by IPython.

[1] https://pypi.python.org/pypi/traitlets

Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com>
---
 package/Config.in                              |  1 +
 package/python-traitlets/Config.in             |  8 ++++++++
 package/python-traitlets/python-traitlets.hash |  2 ++
 package/python-traitlets/python-traitlets.mk   | 18 ++++++++++++++++++
 4 files changed, 29 insertions(+)
 create mode 100644 package/python-traitlets/Config.in
 create mode 100644 package/python-traitlets/python-traitlets.hash
 create mode 100644 package/python-traitlets/python-traitlets.mk

Comments

Thomas Petazzoni April 26, 2017, 7:26 a.m. UTC | #1
Hello,

On Tue, 25 Apr 2017 12:14:16 -0700, Andrey Smirnov wrote:
> diff --git a/package/python-traitlets/Config.in b/package/python-traitlets/Config.in
> new file mode 100644
> index 0000000..ba79218
> --- /dev/null
> +++ b/package/python-traitlets/Config.in
> @@ -0,0 +1,8 @@
> +config BR2_PACKAGE_PYTHON_TRAITLETS
> +	select BR2_PACKAGE_PYTHON_DECORATOR
> +	select BR2_PACKAGE_PYTHON_ENUM if BR2_PACKAGE_PYTHON
> +	bool "python-traitlets"

The "bool" should be before the selects. Have you run your packages
through support/scripts/check-package ?

> +PYTHON_TRAITLETS_DEPENDENCIES = python-decorator
> +
> +ifeq ($(BR2_PACKAGE_PYTHON),y)
> +PYTHON_TRAITLETS_DEPENDENCIES += python-enum
> +endif

Are these really build time dependencies, or just runtime dependencies?

Best regards,

Thomas
Andrey Smirnov April 26, 2017, 6:47 p.m. UTC | #2
On Wed, Apr 26, 2017 at 12:26 AM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Hello,
>
> On Tue, 25 Apr 2017 12:14:16 -0700, Andrey Smirnov wrote:
>> diff --git a/package/python-traitlets/Config.in b/package/python-traitlets/Config.in
>> new file mode 100644
>> index 0000000..ba79218
>> --- /dev/null
>> +++ b/package/python-traitlets/Config.in
>> @@ -0,0 +1,8 @@
>> +config BR2_PACKAGE_PYTHON_TRAITLETS
>> +     select BR2_PACKAGE_PYTHON_DECORATOR
>> +     select BR2_PACKAGE_PYTHON_ENUM if BR2_PACKAGE_PYTHON
>> +     bool "python-traitlets"
>
> The "bool" should be before the selects. Have you run your packages
> through support/scripts/check-package ?

No, forgot to do that, my bad, sorry. Will do in v2.

>
>> +PYTHON_TRAITLETS_DEPENDENCIES = python-decorator
>> +
>> +ifeq ($(BR2_PACKAGE_PYTHON),y)
>> +PYTHON_TRAITLETS_DEPENDENCIES += python-enum
>> +endif
>
> Are these really build time dependencies, or just runtime dependencies?

I think all of those are runtime dependencies. Will remove code like
this for non build time dependencies globally in v2.

Thanks,
Andrey Smirnov
diff mbox

Patch

diff --git a/package/Config.in b/package/Config.in
index eb6d197..47f43eb 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -843,6 +843,7 @@  menu "External python modules"
 	source "package/python-tomako/Config.in"
 	source "package/python-toml/Config.in"
 	source "package/python-tornado/Config.in"
+	source "package/python-traitlets/Config.in"
 	source "package/python-treq/Config.in"
 	source "package/python-twisted/Config.in"
 	source "package/python-txaio/Config.in"
diff --git a/package/python-traitlets/Config.in b/package/python-traitlets/Config.in
new file mode 100644
index 0000000..ba79218
--- /dev/null
+++ b/package/python-traitlets/Config.in
@@ -0,0 +1,8 @@ 
+config BR2_PACKAGE_PYTHON_TRAITLETS
+	select BR2_PACKAGE_PYTHON_DECORATOR
+	select BR2_PACKAGE_PYTHON_ENUM if BR2_PACKAGE_PYTHON
+	bool "python-traitlets"
+	help
+	  A configuration system for Python applications.
+
+	  https://pypi.python.org/pypi/traitlets
diff --git a/package/python-traitlets/python-traitlets.hash b/package/python-traitlets/python-traitlets.hash
new file mode 100644
index 0000000..3b14991
--- /dev/null
+++ b/package/python-traitlets/python-traitlets.hash
@@ -0,0 +1,2 @@ 
+# sha256 locally computed
+sha256 370f938ad730d52272ef74f96f831cb21138f6168e46fe582fe256c35cc656ce python-traitlets-4.3.2.tar.gz
diff --git a/package/python-traitlets/python-traitlets.mk b/package/python-traitlets/python-traitlets.mk
new file mode 100644
index 0000000..6cfd3e5
--- /dev/null
+++ b/package/python-traitlets/python-traitlets.mk
@@ -0,0 +1,18 @@ 
+################################################################################
+#
+# python-traitlets
+#
+################################################################################
+
+PYTHON_TRAITLETS_VERSION = 4.3.2
+PYTHON_TRAITLETS_SOURCE = python-traitlets-$(PYTHON_TRAITLETS_VERSION).tar.gz
+PYTHON_TRAITLETS_SITE = $(call github,ipython,traitlets,$(PYTHON_TRAITLETS_VERSION))
+PYTHON_TRAITLETS_LICENSE = BSD-3c
+PYTHON_TRAITLETS_SETUP_TYPE = distutils
+PYTHON_TRAITLETS_DEPENDENCIES = python-decorator
+
+ifeq ($(BR2_PACKAGE_PYTHON),y)
+PYTHON_TRAITLETS_DEPENDENCIES += python-enum
+endif
+
+$(eval $(python-package))