Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Does not build with GNU Radio 3.9 #95

Open
rgov opened this issue Jan 21, 2021 · 6 comments
Open

Does not build with GNU Radio 3.9 #95

rgov opened this issue Jan 21, 2021 · 6 comments

Comments

@rgov
Copy link

rgov commented Jan 21, 2021

GNU Radio 3.9 moved from SWIG to PyBind11, which might explain this build error (on macOS):

==> cmake .. -DPYTHON_EXECUTABLE=/usr/local/Cellar/gr-iio/HEAD-d0a4adc/libexec/venv/bin/python
Last 15 lines from /Users/rzg/Library/Logs/Homebrew/gr-iio/02.cmake:
-- Found BISON: /usr/local/opt/bison/bin/bison (found suitable version "3.7.4", minimum required is "3.0.2") 
-- Found SWIG: /usr/local/bin/swig (found version "4.0.2")  
-- Found PythonLibs: /usr/local/opt/python@3.9/Frameworks/Python.framework/Versions/3.9/lib/libpython3.9.dylib (found version "3.9.1") 
CMake Error at swig/CMakeLists.txt:36 (include):
  include could not find load file:

    GrSwig


CMake Error at swig/CMakeLists.txt:47 (GR_SWIG_MAKE):
  Unknown CMake command "GR_SWIG_MAKE".
@rgov
Copy link
Author

rgov commented Jan 21, 2021

GNU Radio 3.9 OOT Module Porting Guide

Commit that removed SWIG from GNU Radio: gnuradio/gnuradio@9f60861

@adamhorden
Copy link

adamhorden commented Jan 21, 2021

I have this fixed in:

analogdevicesinc/gnuradio#4

💥

This needs to be merged into gr-iio. I created the PR on the GNU Radio fork as IIO was supposed to be upstreamed in GNU Radio release v3.9.0.0 but this has not happened.

The changes from that need to be merged into gr-iio as the OOT module.

I will create a PR to bring my changes across 🤓 .

@rgov
Copy link
Author

rgov commented Jan 21, 2021

Awesome. Do you know what is going on with master vs upgrade-3.8 here?

@adamhorden
Copy link

adamhorden commented Jan 22, 2021

@rgov I am not sure on that. I am just an end user, using GNU Radio with Analog Devices Plutos.

I have started a work in progress pull request here 🤓 :

#96

@anilgurses
Copy link

Is there any update?

@adamhorden
Copy link

@anilgurses this has been superseded by:

gnuradio/gnuradio#4277

We are currently working on bringing gr-iio in tree. If you compiled that branch from source you would get gr-iio support. We are getting closer every day now to merging this in. Lots of work has happened in the feature/gr-iio branch here 🤓 :

https://github.com/gnuradio/gnuradio/tree/feature/gr-iio

Adam Horden

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants