<div dir="ltr"><div>My motivation was to make PyBOMBS installs work, but I agree that this would be better addressed in GNU Radio itself. I'll just patch these locally until that happens.<br></div><div><br></div>- Clayton<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Feb 8, 2020 at 9:54 AM Sylvain Munaut <<a href="mailto:tnt@246tnt.com">tnt@246tnt.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Nope sorry, not merging those.<br>
<br>
I don't want at any point in the code something that says I am "3.9<br>
compatible" when the actual 3.9 API spec isn't fixed / definitive ...<br>
Because that could yield in the future (says when 3.9 is released and<br>
contains incompatible changes) that you have a git version that<br>
pretends to be 3.9 compatible ... but doesn't actually build against<br>
3.9 ...<br>
<br>
IMHO the change should be in the gnuradio tree itself where the<br>
GnuraidoConfig cmake version check should be ignored (or accept 3.8)<br>
for git dev versions.<br>
That's something that was discussed / brought up at the hackfest last week.<br>
<br>
Cheers,<br>
<br>
     Sylvain<br>
<br>
<br>
<br>
<br>
On Sat, Feb 8, 2020 at 3:48 PM <<a href="mailto:argilo@gmail.com" target="_blank">argilo@gmail.com</a>> wrote:<br>
><br>
> From: Clayton Smith <<a href="mailto:argilo@gmail.com" target="_blank">argilo@gmail.com</a>><br>
><br>
> Currently gr-osmosdr does not build with GNU Radio's master branch,<br>
> which declares itself to be version 3.9. This change allows both 3.8<br>
> and 3.9.<br>
> ---<br>
>  CMakeLists.txt | 8 +++++++-<br>
>  1 file changed, 7 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/CMakeLists.txt b/CMakeLists.txt<br>
> index 56c1a6b185..f3186dee0c 100644<br>
> --- a/CMakeLists.txt<br>
> +++ b/CMakeLists.txt<br>
> @@ -42,7 +42,13 @@ set(CMAKE_BUILD_TYPE ${CMAKE_BUILD_TYPE} CACHE STRING "")<br>
>  list(INSERT CMAKE_MODULE_PATH 0 ${CMAKE_SOURCE_DIR}/cmake/Modules)<br>
><br>
>  # Find GNURadio (pmt and runtime are core, always included)<br>
> -find_package(Gnuradio "3.8" REQUIRED COMPONENTS blocks fft filter)<br>
> +find_package(Gnuradio "3.8" COMPONENTS blocks fft filter)<br>
> +if(NOT Gnuradio_FOUND)<br>
> +    find_package(Gnuradio "3.9" COMPONENTS blocks fft filter)<br>
> +    if(NOT Gnuradio_FOUND)<br>
> +        message(FATAL_ERROR "Unable to find GNURadio")<br>
> +    ENDIF(NOT Gnuradio_FOUND)<br>
> +ENDIF(NOT Gnuradio_FOUND)<br>
><br>
>  # Set the version information here<br>
>  set(VERSION_MAJOR 0)<br>
> --<br>
> 2.17.1<br>
><br>
</blockquote></div>