#53042 closed defect (fixed)
gnuradio @3.7.10.1_1 install fails on Snow Leopard due to undeclared long long math prototypes - and fix
Reported by: | kencu (Ken) | Owned by: | michaelld (Michael Dickens) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | snowleopard | Cc: | |
Port: | gnuradio |
Description
building gnuradio on 10.6.8 hits a pause when certain long long math prototypes are not found, like this:
error: use of undeclared identifier 'llrintf'
the problem appears to be due to several missing long long math prototypes previously identified and fixed by Jeremy in other ports (like clang-3.8). See 52130 and <https://github.com/jeremyhu/libcxx/commit/720feba4874d4ca3131753dc31af127e3c509c36>.
Indeed, adding in that same block of missing prototypes from Jeremy's libcxx github commit fixes the current gnuradio and the build proceeds successfully and gnuradio appears to run.
The block should have gone into this file:
gnuradio-3.7.10.1/gnuradio-runtime/include/gnuradio/math.h
but for some reason that didn't work immediately for me. Might have been something stupid on my end, like pasting it in but not writing to disk before I compiled again.
Anyway, I stuck it here
gnuradio-3.7.10.1/gr-blocks/lib/float_array_to_int.cc
and it worked fine. Haven't hooked it up to a radio yet, but the windows open and the waterfall and other displays come up.
Thanks for gnuradio. My ham license should be more fun with some of those new USB radios to play with.
I might be the only person on earth who cares to run gnuradio on 10.6.8. Do you want a patch and some further fiddling to to see if the math.h fix can be made to work correctly, or should we just leave this on trac for interested persons to find?
Change History (8)
comment:1 Changed 8 years ago by michaelld (Michael Dickens)
comment:2 Changed 8 years ago by ryandesign (Ryan Carsten Schmidt)
Cc: | michaelld removed |
---|---|
Owner: | set to michaelld |
Status: | new → assigned |
comment:4 Changed 8 years ago by michaelld (Michael Dickens)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:5 Changed 8 years ago by michaelld (Michael Dickens)
Please test & verify that this fix works. It's the same as yours just renamed; hopefully I got the new patch name correct :)
comment:6 Changed 8 years ago by kencu (Ken)
looks good
gnuradio-devel @20161210_0+docs+grc+jack+portaudio+qtgui+sdl+swig+uhd+wavelet+wxgui (active) platform='darwin 10' archs='x86_64'
and this
/opt/local/share/gnuradio/examples/qt-gui/display_qt
brings up a nice window with real-looking data displayed.
To get it to build, py27-wxpython-3.0 also has to build (with the default options, at least). With libc++ enabled on 10.6, that currently requires a fix such as 53040 which looks like it is going to be fixed in the main repo shortly.
Thanks. K.
I would welcome a patch, yes! More compatibility is better. Let's take this off-MP ... email me directly at michael.dickens@… . Thx!