[Buildroot] [PATCH 1/1] package/gr-osmosdr: add host-python-six dependency needed for generating Python bindings

[Buildroot] [PATCH 1/1] package/gr-osmosdr: add host-python-six dependency needed for generating Python bindings

[friedtj]

Compiling gr-osmosdr when Python3 support is active fails with

ModuleNotFoundError: No module named 'six' 
make[3]: *** [python/bindings/CMakeFiles/osmosdr_docstrings.dir/build.make:73: python/bindings/docstring_status] Error 1

when generating the Python bindings. The following patch solves the problem.


Signed-off-by: Jean-Michel Friedt <jmfriedt@femto-st.fr>
---
 package/gr-osmosdr/gr-osmosdr.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/gr-osmosdr/gr-osmosdr.mk b/package/gr-osmosdr/gr-osmosdr.mk
index 8491c005d8..316ed80887 100644
--- a/package/gr-osmosdr/gr-osmosdr.mk
+++ b/package/gr-osmosdr/gr-osmosdr.mk
@@ -24,7 +24,7 @@ GR_OSMOSDR_INSTALL_STAGING = YES
 
 ifeq ($(BR2_PACKAGE_GNURADIO_PYTHON),y)
 GR_OSMOSDR_CONF_OPTS += -DENABLE_PYTHON=ON
-GR_OSMOSDR_DEPENDENCIES += python3
+GR_OSMOSDR_DEPENDENCIES += python3 host-python-six
 else
 GR_OSMOSDR_CONF_OPTS += -DENABLE_PYTHON=OFF
 endif
-- 
2.43.0
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 1/1] package/gr-osmosdr: add host-python-six dependency needed for generating Python bindings

[Thomas Petazzoni via buildroot]

Hello Jean-Michel,

On Thu, 11 Apr 2024 14:07:43 +0200 (CEST)
friedtj@free.fr wrote:

> Compiling gr-osmosdr when Python3 support is active fails with
> 
> ModuleNotFoundError: No module named 'six' 
> make[3]: *** [python/bindings/CMakeFiles/osmosdr_docstrings.dir/build.make:73: python/bindings/docstring_status] Error 1
> 
> when generating the Python bindings. The following patch solves the problem.
> 
> 
> Signed-off-by: Jean-Michel Friedt <jmfriedt@femto-st.fr>
> ---
>  package/gr-osmosdr/gr-osmosdr.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Thanks for the patch. However, I have applied a similar patch which was
submitted earlier by Martin Böh:

  https://gitlab.com/buildroot.org/buildroot/-/commit/cfda1f0b87ebfb7686b82f319c531d4d28fdfd67

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot