diff mbox series

[1/2] package/python-scipy: fix io runtime dependency

Message ID 20220802173330.15176-1-guillaume.bressaix@gmail.com
State Accepted
Headers show
Series [1/2] package/python-scipy: fix io runtime dependency | expand

Commit Message

Guillaume Bres Aug. 2, 2022, 5:33 p.m. UTC
We need python built with the --zlib feature, for the io
module to import properly and be usable

Signed-off-by: Guillaume W. Bres <guillaume.bressaix@gmail.com>
---
I did not provide the correct BR flag here. Let's test this module anyway
with the runtime test (see the following commit).
It turns out the scipy.io module is the only one that does not work right away
---
 package/python-scipy/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Thomas Petazzoni Aug. 3, 2022, 4:36 p.m. UTC | #1
On Tue,  2 Aug 2022 19:33:29 +0200
"Guillaume W. Bres" <guillaume.bressaix@gmail.com> wrote:

> We need python built with the --zlib feature, for the io
> module to import properly and be usable
> 
> Signed-off-by: Guillaume W. Bres <guillaume.bressaix@gmail.com>
> ---
> I did not provide the correct BR flag here. Let's test this module anyway
> with the runtime test (see the following commit).
> It turns out the scipy.io module is the only one that does not work right away
> ---
>  package/python-scipy/Config.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Both applied, with slightly reworked commit logs/titles. Thanks!

Thomas
diff mbox series

Patch

diff --git a/package/python-scipy/Config.in b/package/python-scipy/Config.in
index 0fa6c9e311..229b5997ad 100644
--- a/package/python-scipy/Config.in
+++ b/package/python-scipy/Config.in
@@ -6,7 +6,7 @@  config BR2_PACKAGE_PYTHON_SCIPY
 	depends on BR2_TOOLCHAIN_HAS_FORTRAN # lapack
 	depends on BR2_INSTALL_LIBSTDCPP
 	select BR2_PACKAGE_LAPACK
-	select BR2_PACKAGE_ZLIB # runtime scipy::io
+	select BR2_PACKAGE_PYTHON3_ZLIB # runtime scipy::io
 	select BR2_PACKAGE_PYTHON_NUMPY # runtime
 	select BR2_PACKAGE_PYTHON_PYBIND
 	help