discuss-gnuradio
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Discuss-gnuradio] GR standard UHD options parser module (was UHD de


From: Tom Rondeau
Subject: Re: [Discuss-gnuradio] GR standard UHD options parser module (was UHD default subdevice.)
Date: Fri, 30 Dec 2011 18:51:32 -0500

On Fri, Dec 30, 2011 at 6:40 PM, Josh Blum <address@hidden> wrote:


On 12/30/2011 03:12 PM, LRK wrote:
> On Fri, Dec 30, 2011 at 12:41:48PM -0800, Josh Blum wrote:
>>
>> And please, tell us the errors you get on FreeBSD. They may be simple or
>> easily fixable. Let us know!
>
>
> I am rebuilding today on my FreeBSD 8.2 machine, Using gmake.
> Same hardware runs Ubuntu 11.04 :
>
> gr_udp_sink.cc: In constructor 'gr_udp_sink::gr_udp_sink(size_t, const char*, sh
> ort unsigned int, int, bool)':
> gr_udp_sink.cc:112: error: 'IPPROTO_UDP' was not declared in this scope
>
>
> gr_udp_source.cc has lines that are not in the sink. I think these were
> there at one time but vanished. I fix this kind of thing with a patch until
> it is fixed in the source. ( I'll do that and proceede.)
>
>  typedef void* optval_t;
> +
> +// ntohs() on FreeBSD may require both netinet/in.h and arpa/inet.h, in order
> +#if defined(HAVE_NETINET_IN_H)
> +#include <netinet/in.h>
> +#endif
> +#if defined(HAVE_ARPA_INET_H)
> +#include <arpa/inet.h>
> +#endif
> +
>  #elif defined(HAVE_WINDOWS_H)
>
>
>

Boost fan here. I think replacing that with #include <boost/asio.hpp>
would take care of all those system-specific intricacies.

Here are some blocks using boost asio. If anyone felt motivated to port
this into the existing udp_source/sink blocks, that would drastically
clean up the code:

http://gnuradio.org/cgit/jblum.git/tree/gnuradio-core/src/lib/io/gr_socket_to_blob.cc?h=msg_passing_squashed
http://gnuradio.org/cgit/jblum.git/tree/gnuradio-core/src/lib/io/gr_blob_to_socket.cc?h=msg_passing_squashed

-Josh

I think the OSX community has similar issues with that block, too. Anyone know more about that?

Hopefully, we can move to the Boost ASIO to fix this for cross platform issues like this.

Tom
 

reply via email to

[Prev in Thread] Current Thread [Next in Thread]