discuss-gnuradio
[Top][All Lists]
Advanced

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

Re: [Discuss-gnuradio] New structural change


From: Philip Balister
Subject: Re: [Discuss-gnuradio] New structural change
Date: Mon, 19 Dec 2011 10:40:31 -0500
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.24) Gecko/20111108 Fedora/3.1.16-1.fc14 Thunderbird/3.1.16

On 12/18/2011 08:30 PM, Michael Dickens wrote:
> On Dec 18, 2011, at 3:41 PM, Tom Rondeau wrote:
>> The differences, though, coming in the naming scheme and installation
>> method. Instead of having a gr_<name>.h file installed into
>> $prefix/include/gnuradio, it's just <name>.h that is installed into
>> $prefix/include/gnuradio/blocks. Likewise, the headers in gr-filter are
>> installed into $prefix/include/gnuradio/filter.
> 
> I like this change in header include structure.  Most of the projects I work 
> on do it this way, and it makes a lot of sense.  Because this will be a 
> change that impacts just about everyone doing C++ development using GNU 
> Radio, I think -if possible- it would be good to keep the "gr_<name>.h" 
> headers around, but all they do is 2 things:
> 
> 1) print out a #warning about being deprecated, and to instead use 
> "blocks/<name>.h" -- or, whatever makes sense for such a message; and
> 
> 2) #include "blocks/<name>.h" so that the user's code still compiles as before
> 
> That way, we benefit from the new header include structure, but aren't 
> totally screwed if we've forgotten to use it.  And, somewhere down the road 
> after an appropriate amount of time, the "gr_<name>.h" headers can be removed 
> entirely, if desired, because everyone has been well warned.  I hope 
> something along this line can be done.

At the same time, can we clean up the various combination of
-I/usr/inclide/gnuradio etc

with

#include "gr-header.h"
#include "gnuradio/gr-header.h"

?

I'd like to see the use of -I dropped where possible and all header
files referred to relative to /usr/include (or /usr/local/include)

Philip



reply via email to

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