Re: [Buildroot] [PATCH v2] package/gnuradio: bump version to 3.10.3.0

[Thomas Petazzoni <thomas.petazzoni@bootlin.com>]

Hello,

On Fri, 16 Sep 2022 15:04:23 +0200
Gwenhael Goavec-Merou <gwenj@trabucayre.com> wrote:

> diff --git a/package/gnuradio/Config.in b/package/gnuradio/Config.in
> index 103175946c..fd693c1851 100644
> --- a/package/gnuradio/Config.in
> +++ b/package/gnuradio/Config.in
> @@ -15,19 +15,19 @@ config BR2_PACKAGE_GNURADIO
>  	depends on BR2_TOOLCHAIN_HAS_THREADS_NPTL
>  	depends on BR2_USE_MMU # use fork()
>  	depends on BR2_USE_WCHAR # boost
> -	depends on BR2_TOOLCHAIN_SUPPORTS_ALWAYS_LOCKFREE_ATOMIC_INTS # boost-atomic, boost-filesystem
> +	depends on BR2_TOOLCHAIN_SUPPORTS_ALWAYS_LOCKFREE_ATOMIC_INTS # boost-atomic, boost-thread
>  	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_43744
>  	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_64735 # boost-thread
>  	select BR2_PACKAGE_BOOST
>  	select BR2_PACKAGE_BOOST_ATOMIC
>  	select BR2_PACKAGE_BOOST_DATE_TIME
> -	select BR2_PACKAGE_BOOST_FILESYSTEM
>  	select BR2_PACKAGE_BOOST_PROGRAM_OPTIONS
>  	select BR2_PACKAGE_BOOST_REGEX
>  	select BR2_PACKAGE_BOOST_SYSTEM
>  	select BR2_PACKAGE_BOOST_THREAD
>  	select BR2_PACKAGE_LOG4CPP
>  	select BR2_PACKAGE_GMP
> +	select BR2_PACKAGE_SPDLOG
>  	select BR2_PACKAGE_VOLK
>  	help
>  	  GNU Radio is a free & open-source software development
> @@ -80,7 +80,8 @@ config BR2_PACKAGE_GNURADIO_PYTHON
>  	depends on BR2_PACKAGE_PYTHON_NUMPY_ARCH_SUPPORTS
>  	depends on BR2_TOOLCHAIN_USES_GLIBC || BR2_TOOLCHAIN_USES_MUSL # python-numpy
>  	select BR2_PACKAGE_BOOST_PYTHON
> -	select BR2_PACKAGE_PYTHON_NUMPY # runtime
> +	select BR2_PACKAGE_PYTHON_NUMPY

Why are you removing the "# runtime" comment here? Your patch doesn't
change numpy to a build time dependency.

> +	select BR2_PACKAGE_PYTHON_PYBIND

The target pybind is a build time dependency?

> -# host-python-mako and host-python-six are needed for volk to compile
>  GNURADIO_DEPENDENCIES = \
>  	host-python3 \
> -	host-python-mako \
> -	host-python-six \
> -	host-swig \
>  	boost \
>  	log4cpp \
>  	gmp \
> +	spdlog \
>  	volk
>  
>  GNURADIO_CONF_OPTS = \
> @@ -115,7 +112,8 @@ GNURADIO_CONF_OPTS += -DENABLE_GR_FILTER=OFF
>  endif
>  
>  ifeq ($(BR2_PACKAGE_GNURADIO_PYTHON),y)
> -GNURADIO_DEPENDENCIES += python3
> +GNURADIO_DEPENDENCIES += python3 python-pybind \
> +	host-python-numpy host-python-packaging

Could you clarify if you need python-pybind on the target or on the
host? And also explain in the commit log why host-python-numpy and
host-python-packaging are now needed?

Thanks!

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