[PATCH] build: Allow building with GNU Radio master

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/osmocom-sdr@lists.osmocom.org/.

Sylvain Munaut tnt at 246tnt.com
Sat Feb 8 14:55:25 UTC 2020


Nope sorry, not merging those.

I don't want at any point in the code something that says I am "3.9
compatible" when the actual 3.9 API spec isn't fixed / definitive ...
Because that could yield in the future (says when 3.9 is released and
contains incompatible changes) that you have a git version that
pretends to be 3.9 compatible ... but doesn't actually build against
3.9 ...

IMHO the change should be in the gnuradio tree itself where the
GnuraidoConfig cmake version check should be ignored (or accept 3.8)
for git dev versions.
That's something that was discussed / brought up at the hackfest last week.

Cheers,

     Sylvain




On Sat, Feb 8, 2020 at 3:48 PM <argilo at gmail.com> wrote:
>
> From: Clayton Smith <argilo at gmail.com>
>
> Currently gr-osmosdr does not build with GNU Radio's master branch,
> which declares itself to be version 3.9. This change allows both 3.8
> and 3.9.
> ---
>  CMakeLists.txt | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/CMakeLists.txt b/CMakeLists.txt
> index 56c1a6b185..f3186dee0c 100644
> --- a/CMakeLists.txt
> +++ b/CMakeLists.txt
> @@ -42,7 +42,13 @@ set(CMAKE_BUILD_TYPE ${CMAKE_BUILD_TYPE} CACHE STRING "")
>  list(INSERT CMAKE_MODULE_PATH 0 ${CMAKE_SOURCE_DIR}/cmake/Modules)
>
>  # Find GNURadio (pmt and runtime are core, always included)
> -find_package(Gnuradio "3.8" REQUIRED COMPONENTS blocks fft filter)
> +find_package(Gnuradio "3.8" COMPONENTS blocks fft filter)
> +if(NOT Gnuradio_FOUND)
> +    find_package(Gnuradio "3.9" COMPONENTS blocks fft filter)
> +    if(NOT Gnuradio_FOUND)
> +        message(FATAL_ERROR "Unable to find GNURadio")
> +    ENDIF(NOT Gnuradio_FOUND)
> +ENDIF(NOT Gnuradio_FOUND)
>
>  # Set the version information here
>  set(VERSION_MAJOR 0)
> --
> 2.17.1
>



More information about the osmocom-sdr mailing list