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: Josh Blum
Subject: Re: [Discuss-gnuradio] GR standard UHD options parser module (was UHD default subdevice.)
Date: Fri, 30 Dec 2011 15:40:03 -0800
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:8.0) Gecko/20111124 Thunderbird/8.0


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



reply via email to

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