diff mbox

[v2,05/10] package/python-scandir: New package

Message ID 20170427213340.26917-6-andrew.smirnov@gmail.com
State Changes Requested
Headers show

Commit Message

Andrey Smirnov April 27, 2017, 9:33 p.m. UTC
Add 'scandir'[1] package to buildroot. Needed by 'pathlib2'.

[1] https://pypi.python.org/pypi/scandir
Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com>
---
 package/Config.in                          |  1 +
 package/python-scandir/Config.in           |  6 ++++++
 package/python-scandir/python-scandir.hash |  2 ++
 package/python-scandir/python-scandir.mk   | 13 +++++++++++++
 4 files changed, 22 insertions(+)
 create mode 100644 package/python-scandir/Config.in
 create mode 100644 package/python-scandir/python-scandir.hash
 create mode 100644 package/python-scandir/python-scandir.mk

Comments

Thomas Petazzoni May 4, 2017, 8:18 p.m. UTC | #1
Hello,

On Thu, 27 Apr 2017 14:33:35 -0700, Andrey Smirnov wrote:

> +config BR2_PACKAGE_PYTHON_SCANDIR
> +	bool "python-scandir"
> +	help
> +	  A configuration system for Python applications.

This description is bogus.

> +	  https://pypi.python.org/pypi/pickleshare

This URL is wrong.

And the entry to DEVELOPERS file is missing.

Could you please check your entire series, and verify all those
details? Once they are all fixed, could you resubmit an updated version?

Please verify the dependencies and licenses as well.

I'll mark the series as Changes Requested in patchwork in the mean
time. I have only applied the python-decorator and python-simplegeneric
packages.

Thanks!

Thomas
Andrey Smirnov May 4, 2017, 8:27 p.m. UTC | #2
On Thu, May 4, 2017 at 1:18 PM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Hello,
>
> On Thu, 27 Apr 2017 14:33:35 -0700, Andrey Smirnov wrote:
>
>> +config BR2_PACKAGE_PYTHON_SCANDIR
>> +     bool "python-scandir"
>> +     help
>> +       A configuration system for Python applications.
>
> This description is bogus.
>
>> +       https://pypi.python.org/pypi/pickleshare
>
> This URL is wrong.
>
> And the entry to DEVELOPERS file is missing.
>
> Could you please check your entire series, and verify all those
> details? Once they are all fixed, could you resubmit an updated version?

Sorry about that, my bad. I'll re-check the series and submit v3.

>
> Please verify the dependencies and licenses as well.

Will do.

Thanks,
Andrey Smirnov
diff mbox

Patch

diff --git a/package/Config.in b/package/Config.in
index 143bccc..c64105e 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -823,6 +823,7 @@  menu "External python modules"
 	source "package/python-rpi-gpio/Config.in"
 	source "package/python-rtslib-fb/Config.in"
 	source "package/python-scapy3k/Config.in"
+	source "package/python-scandir/Config.in"
 	source "package/python-sdnotify/Config.in"
 	source "package/python-serial/Config.in"
 	source "package/python-service-identity/Config.in"
diff --git a/package/python-scandir/Config.in b/package/python-scandir/Config.in
new file mode 100644
index 0000000..5e8ca0d
--- /dev/null
+++ b/package/python-scandir/Config.in
@@ -0,0 +1,6 @@ 
+config BR2_PACKAGE_PYTHON_SCANDIR
+	bool "python-scandir"
+	help
+	  A configuration system for Python applications.
+
+	  https://pypi.python.org/pypi/pickleshare
diff --git a/package/python-scandir/python-scandir.hash b/package/python-scandir/python-scandir.hash
new file mode 100644
index 0000000..a7ee337
--- /dev/null
+++ b/package/python-scandir/python-scandir.hash
@@ -0,0 +1,2 @@ 
+# sha256 locally computed
+sha256 c2612d1a487d80fb4701b4a91ca1b8f8a695b1ae820570815e85e8c8b23f1283 scandir-1.5.tar.gz
diff --git a/package/python-scandir/python-scandir.mk b/package/python-scandir/python-scandir.mk
new file mode 100644
index 0000000..cf921ae
--- /dev/null
+++ b/package/python-scandir/python-scandir.mk
@@ -0,0 +1,13 @@ 
+################################################################################
+#
+# python-scandir
+#
+################################################################################
+
+PYTHON_SCANDIR_VERSION = 1.5
+PYTHON_SCANDIR_SOURCE = scandir-$(PYTHON_SCANDIR_VERSION).tar.gz
+PYTHON_SCANDIR_SITE = https://pypi.python.org/packages/bd/f4/3143e0289faf0883228017dbc6387a66d0b468df646645e29e1eb89ea10e
+PYTHON_SCANDIR_LICENSE = BSD-3-Clause
+PYTHON_SCANDIR_SETUP_TYPE = distutils
+
+$(eval $(python-package))