Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#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)

I would welcome a patch, yes! More compatibility is better. Let's take this off-MP ... email me directly at michael.dickens@… . Thx!

comment:2 Changed 8 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: michaelld removed
Owner: set to michaelld
Status: newassigned

comment:3 Changed 8 years ago by michaelld (Michael Dickens)

In 94b9570e/macports-ports:

gnuradio: update devel to 0ec18440 (20161210); remove $id$ line; add patch to fix building on 10.6 Snow Leopard due to missing function prototype, which addresses ticket #53042 .

comment:4 Changed 8 years ago by michaelld (Michael Dickens)

Resolution: fixed
Status: assignedclosed

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.

comment:7 Changed 8 years ago by michaelld (Michael Dickens)

Great! Thx for reporting back.

comment:8 Changed 8 years ago by michaelld (Michael Dickens)

In b7a93dba/macports-ports:

gnuradio: update devel to 0ec18440 (20161210); remove $id$ line; add patch to fix building on 10.6 Snow Leopard due to missing function prototype, which addresses ticket #53042 .

Note: See TracTickets for help on using tickets.