[Buildroot] [PATCH v3 1/2] package/python-kiwisolver: new package

[Buildroot] [PATCH v3 1/2] package/python-kiwisolver: new package

[Esben Haabendal]

Signed-off-by: Esben Haabendal <esben@geanix.com>
Reviewed-by: Yegor Yefremov <yegorslists@googlemail.com>
---
 DEVELOPERS                                      |  1 +
 package/Config.in                               |  1 +
 package/python-kiwisolver/Config.in             | 17 +++++++++++++++++
 .../python-kiwisolver/python-kiwisolver.hash    |  3 +++
 package/python-kiwisolver/python-kiwisolver.mk  |  7 +++++++
 5 files changed, 29 insertions(+)
 create mode 100644 package/python-kiwisolver/Config.in
 create mode 100644 package/python-kiwisolver/python-kiwisolver.hash
 create mode 100644 package/python-kiwisolver/python-kiwisolver.mk

diff --git a/DEVELOPERS b/DEVELOPERS
index 3b1d03c82684..583a65b14ba9 100644
--- a/DEVELOPERS
+++ b/DEVELOPERS
@@ -719,6 +719,7 @@ F:	package/szip/
 
 N:	Esben Haabendal <esben@haabendal.dk>
 F:	boot/gummiboot/
+F:	package/python-kiwisolver/
 
 N:	Etienne Carriere <etienne.carriere@linaro.org>
 F:	boot/optee-os/
diff --git a/package/Config.in b/package/Config.in
index 9336261c5d9e..dddce6636168 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -933,6 +933,7 @@ menu "External python modules"
 	source "package/python-jsonmodels/Config.in"
 	source "package/python-jsonschema/Config.in"
 	source "package/python-keyring/Config.in"
+	source "package/python-kiwisolver/Config.in"
 	source "package/python-libconfig/Config.in"
 	source "package/python-libusb1/Config.in"
 	source "package/python-lmdb/Config.in"
diff --git a/package/python-kiwisolver/Config.in b/package/python-kiwisolver/Config.in
new file mode 100644
index 000000000000..6d31fdc35787
--- /dev/null
+++ b/package/python-kiwisolver/Config.in
@@ -0,0 +1,17 @@
+config BR2_PACKAGE_PYTHON_KIWISOLVER
+	bool "python-kiwisolver"
+	depends on BR2_INSTALL_LIBSTDCPP
+	depends on BR2_PACKAGE_PYTHON || BR2_PACKAGE_PYTHON
+	help
+	  Kiwi is an efficient C++ implementation of the Cassowary constraint
+	  solving algorithm. Kiwi is an implementation of the algorithm based
+	  on the seminal Cassowary paper. It is not a refactoring of the
+	  original C++ solver. Kiwi has been designed from the ground up to be
+	  lightweight and fast. Kiwi ranges from 10x to 500x faster than the
+	  original Cassowary solver with typical use cases gaining a 40x
+	  improvement. Memory savings are consistently > 5x.
+
+	  https://github.com/nucleic/kiwi
+
+comment "python-kiwisolver needs a toolchain w/ C++"
+	depends on !BR2_INSTALL_LIBSTDCPP
diff --git a/package/python-kiwisolver/python-kiwisolver.hash b/package/python-kiwisolver/python-kiwisolver.hash
new file mode 100644
index 000000000000..9cdb27002cf2
--- /dev/null
+++ b/package/python-kiwisolver/python-kiwisolver.hash
@@ -0,0 +1,3 @@
+# Locally calculated
+sha256 48168c0ace18319c649e31907f6d8586f648e503560f506f128798b99393bcdb  python-kiwisolver-1.1.0.tar.gz
+sha256 95c32f73d43fa54d8146badbcbd73b83acb7fa2cc81a7cef7784b3cd33b146a0  LICENSE
diff --git a/package/python-kiwisolver/python-kiwisolver.mk b/package/python-kiwisolver/python-kiwisolver.mk
new file mode 100644
index 000000000000..96019fc29328
--- /dev/null
+++ b/package/python-kiwisolver/python-kiwisolver.mk
@@ -0,0 +1,7 @@
+PYTHON_KIWISOLVER_VERSION = 1.1.0
+PYTHON_KIWISOLVER_SITE = $(call github,nucleic,kiwi,$(PYTHON_KIWISOLVER_VERSION))
+PYTHON_KIWISOLVER_LICENSE = BSD-3-Clause
+PYTHON_KIWISOLVER_LICENSE_FILES = LICENSE
+PYTHON_KIWISOLVER_SETUP_TYPE = setuptools
+
+$(eval $(python-package))
-- 
2.23.0

[Buildroot] [PATCH v3 2/2] package/python-matplotlib: add kiwisolver dependency

[Esben Haabendal]

This is needed by matplotlib.pyplot, and is also documented at
https://matplotlib.org/users/installing.html#building-on-linux

Signed-off-by: Esben Haabendal <esben@geanix.com>
Reviewed-by: Yegor Yefremov <yegorslists@googlemail.com>
---
 package/python-matplotlib/Config.in | 1 +
 1 file changed, 1 insertion(+)

diff --git a/package/python-matplotlib/Config.in b/package/python-matplotlib/Config.in
index 07160affd043..e6ed8cda2f28 100644
--- a/package/python-matplotlib/Config.in
+++ b/package/python-matplotlib/Config.in
@@ -9,6 +9,7 @@ config BR2_PACKAGE_PYTHON_MATPLOTLIB
 	select BR2_PACKAGE_PYTHON_CYCLER
 	select BR2_PACKAGE_PYTHON_SETUPTOOLS # runtime
 	select BR2_PACKAGE_PYTHON_DATEUTIL # runtime
+	select BR2_PACKAGE_PYTHON_KIWISOLVER # runtime
 	select BR2_PACKAGE_PYTHON_NUMPY # runtime
 	select BR2_PACKAGE_PYTHON_PYPARSING # runtime
 	select BR2_PACKAGE_PYTHON3_ZLIB # runtime
-- 
2.23.0

[Buildroot] [PATCH v3 1/2] package/python-kiwisolver: new package

[Yegor Yefremov]

Hi Esben,

On Tue, Sep 17, 2019 at 10:50 AM Esben Haabendal <esben@geanix.com> wrote:
>
> Signed-off-by: Esben Haabendal <esben@geanix.com>
> Reviewed-by: Yegor Yefremov <yegorslists@googlemail.com>
> ---
>  DEVELOPERS                                      |  1 +
>  package/Config.in                               |  1 +
>  package/python-kiwisolver/Config.in             | 17 +++++++++++++++++
>  .../python-kiwisolver/python-kiwisolver.hash    |  3 +++
>  package/python-kiwisolver/python-kiwisolver.mk  |  7 +++++++
>  5 files changed, 29 insertions(+)
>  create mode 100644 package/python-kiwisolver/Config.in
>  create mode 100644 package/python-kiwisolver/python-kiwisolver.hash
>  create mode 100644 package/python-kiwisolver/python-kiwisolver.mk
>
> diff --git a/DEVELOPERS b/DEVELOPERS
> index 3b1d03c82684..583a65b14ba9 100644
> --- a/DEVELOPERS
> +++ b/DEVELOPERS
> @@ -719,6 +719,7 @@ F:  package/szip/
>
>  N:     Esben Haabendal <esben@haabendal.dk>
>  F:     boot/gummiboot/
> +F:     package/python-kiwisolver/
>
>  N:     Etienne Carriere <etienne.carriere@linaro.org>
>  F:     boot/optee-os/
> diff --git a/package/Config.in b/package/Config.in
> index 9336261c5d9e..dddce6636168 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -933,6 +933,7 @@ menu "External python modules"
>         source "package/python-jsonmodels/Config.in"
>         source "package/python-jsonschema/Config.in"
>         source "package/python-keyring/Config.in"
> +       source "package/python-kiwisolver/Config.in"
>         source "package/python-libconfig/Config.in"
>         source "package/python-libusb1/Config.in"
>         source "package/python-lmdb/Config.in"
> diff --git a/package/python-kiwisolver/Config.in b/package/python-kiwisolver/Config.in
> new file mode 100644
> index 000000000000..6d31fdc35787
> --- /dev/null
> +++ b/package/python-kiwisolver/Config.in
> @@ -0,0 +1,17 @@
> +config BR2_PACKAGE_PYTHON_KIWISOLVER
> +       bool "python-kiwisolver"
> +       depends on BR2_INSTALL_LIBSTDCPP
> +       depends on BR2_PACKAGE_PYTHON || BR2_PACKAGE_PYTHON
> +       help
> +         Kiwi is an efficient C++ implementation of the Cassowary constraint
> +         solving algorithm. Kiwi is an implementation of the algorithm based
> +         on the seminal Cassowary paper. It is not a refactoring of the
> +         original C++ solver. Kiwi has been designed from the ground up to be
> +         lightweight and fast. Kiwi ranges from 10x to 500x faster than the
> +         original Cassowary solver with typical use cases gaining a 40x
> +         improvement. Memory savings are consistently > 5x.
> +
> +         https://github.com/nucleic/kiwi
> +
> +comment "python-kiwisolver needs a toolchain w/ C++"
> +       depends on !BR2_INSTALL_LIBSTDCPP
> diff --git a/package/python-kiwisolver/python-kiwisolver.hash b/package/python-kiwisolver/python-kiwisolver.hash
> new file mode 100644
> index 000000000000..9cdb27002cf2
> --- /dev/null
> +++ b/package/python-kiwisolver/python-kiwisolver.hash
> @@ -0,0 +1,3 @@
> +# Locally calculated
> +sha256 48168c0ace18319c649e31907f6d8586f648e503560f506f128798b99393bcdb  python-kiwisolver-1.1.0.tar.gz
> +sha256 95c32f73d43fa54d8146badbcbd73b83acb7fa2cc81a7cef7784b3cd33b146a0  LICENSE
> diff --git a/package/python-kiwisolver/python-kiwisolver.mk b/package/python-kiwisolver/python-kiwisolver.mk
> new file mode 100644
> index 000000000000..96019fc29328
> --- /dev/null
> +++ b/package/python-kiwisolver/python-kiwisolver.mk
> @@ -0,0 +1,7 @@
> +PYTHON_KIWISOLVER_VERSION = 1.1.0
> +PYTHON_KIWISOLVER_SITE = $(call github,nucleic,kiwi,$(PYTHON_KIWISOLVER_VERSION))
> +PYTHON_KIWISOLVER_LICENSE = BSD-3-Clause
> +PYTHON_KIWISOLVER_LICENSE_FILES = LICENSE
> +PYTHON_KIWISOLVER_SETUP_TYPE = setuptools
> +
> +$(eval $(python-package))
> --
> 2.23.0

Could you also mark the previous patch versions as "Superceded" in [1]. Thanks.

[1] https://patchwork.ozlabs.org/project/buildroot/list/?submitter=76514

Yegor

[Buildroot] [PATCH v3 1/2] package/python-kiwisolver: new package

[Esben Haabendal]

Yegor Yefremov <yegorslists@googlemail.com> writes:

> Could you also mark the previous patch versions as "Superceded" in [1]. Thanks.
>
> [1] https://patchwork.ozlabs.org/project/buildroot/list/?submitter=76514

Sure thing.  Done :)

/Esben

[Buildroot] [PATCH v3 1/2] package/python-kiwisolver: new package

[Thomas Petazzoni]

Hello,

On Tue, 17 Sep 2019 10:50:33 +0200
Esben Haabendal <esben@geanix.com> wrote:

> +config BR2_PACKAGE_PYTHON_KIWISOLVER
> +	bool "python-kiwisolver"
> +	depends on BR2_INSTALL_LIBSTDCPP
> +	depends on BR2_PACKAGE_PYTHON || BR2_PACKAGE_PYTHON

a dependency on python || python is not very useful, especially since
anyway this Config.in is already included in a python || python3
condition.

> diff --git a/package/python-kiwisolver/python-kiwisolver.mk b/package/python-kiwisolver/python-kiwisolver.mk
> new file mode 100644
> index 000000000000..96019fc29328
> --- /dev/null
> +++ b/package/python-kiwisolver/python-kiwisolver.mk
> @@ -0,0 +1,7 @@

You forgot our completely useless, but absolutely mandatory and lovely
comment header that we have in all packages.

I fixed both issues and applied to master. Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

[Buildroot] [PATCH v3 1/2] package/python-kiwisolver: new package

[Esben Haabendal]

Thomas Petazzoni <thomas.petazzoni@bootlin.com> writes:

> Hello,
>
> On Tue, 17 Sep 2019 10:50:33 +0200
> Esben Haabendal <esben@geanix.com> wrote:
>
>> +config BR2_PACKAGE_PYTHON_KIWISOLVER
>> +	bool "python-kiwisolver"
>> +	depends on BR2_INSTALL_LIBSTDCPP
>> +	depends on BR2_PACKAGE_PYTHON || BR2_PACKAGE_PYTHON
>
> a dependency on python || python is not very useful, especially since
> anyway this Config.in is already included in a python || python3
> condition.

Ah.  Both did I make a typo in v3, and what I wanted to do did not make
any difference.  The first lesson of today :)

>
>> diff --git a/package/python-kiwisolver/python-kiwisolver.mk b/package/python-kiwisolver/python-kiwisolver.mk
>> new file mode 100644
>> index 000000000000..96019fc29328
>> --- /dev/null
>> +++ b/package/python-kiwisolver/python-kiwisolver.mk
>> @@ -0,0 +1,7 @@
>
> You forgot our completely useless, but absolutely mandatory and lovely
> comment header that we have in all packages.

I will try to remember next time :)

> I fixed both issues and applied to master. Thanks!

Thanks.

/Esben

[Buildroot] [PATCH v3 1/2] package/python-kiwisolver: new package

[Peter Korsgaard]

>>>>> "Esben" == Esben Haabendal <esben@geanix.com> writes:

Hi,

 >> You forgot our completely useless, but absolutely mandatory and lovely
 >> comment header that we have in all packages.

 > I will try to remember next time :)

Hint: We have ./utils/check-package which warns about all these kind of
things.

-- 
Bye, Peter Korsgaard

[Buildroot] [PATCH v3 2/2] package/python-matplotlib: add kiwisolver dependency

[Thomas Petazzoni]

On Tue, 17 Sep 2019 10:50:34 +0200
Esben Haabendal <esben@geanix.com> wrote:

> This is needed by matplotlib.pyplot, and is also documented at
> https://matplotlib.org/users/installing.html#building-on-linux
> 
> Signed-off-by: Esben Haabendal <esben@geanix.com>
> Reviewed-by: Yegor Yefremov <yegorslists@googlemail.com>
> ---
>  package/python-matplotlib/Config.in | 1 +
>  1 file changed, 1 insertion(+)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com