[Discuss-gnuradio] Include guard bug (was: Re: Issue while installing GR 3.7)

2014-02-21 Thread Marcus Müller
Should be :) Just as side info: http://www.boost.org/doc/libs/1_54_0/doc/html/boost/random/mt19937.html Sorry, totally running low on clues here... This is twice as strange since boost::random is missing mt19937; if it was std:: I'd guess on a non-C++11 standard library, but like it is... If

Re: [Discuss-gnuradio] Include guard bug

2014-02-21 Thread Marcus Müller
Ruecan: I got carried away. This is indeed a bugfix for the header file not being processed in some cases, but since the error appeared although actually processing the fixed header file, I've run out of ideas, still. On 02/21/2014 04:37 PM, Marcus Müller wrote: Should be :) Just as side

Re: [Discuss-gnuradio] Include guard bug

2014-02-21 Thread Ruecan
I just pulled the changes then did make but get the same error. I am not so familiar with git. After pulling your bugfix, do I need to make clean, remove the CMakeCache then do cmake again then make or am I missing some part of the process. -- View this message in context:

Re: [Discuss-gnuradio] Include guard bug

2014-02-21 Thread Dan CaJacob
Yeah, start completely clean. Very Respectfully, Dan CaJacob On Fri, Feb 21, 2014 at 11:17 AM, Ruecan naceuram...@gmail.com wrote: I just pulled the changes then did make but get the same error. I am not so familiar with git. After pulling your bugfix, do I need to make clean, remove the

Re: [Discuss-gnuradio] Include guard bug

2014-02-21 Thread Tim
good find, this was probably my fault - sorry We should consider changing headers to use #pragma once which is simpler and less error prone do people still use gcc older than 3.4 ? I think this is pretty widely supported now not sure if that would cause swig issues as well - -Tim On

Re: [Discuss-gnuradio] Include guard bug

2014-02-21 Thread Ruecan
Guys, I re make from clean but still got the same error. PS: after pulling the bugfix, do I need to execute any other command, other than *git pull https://github.com/marcusmueller/gnuradio.git master_fix_message_strobe_random_ifndef * Tim, I did not get what you mean by Tim O'Shea wrote

Re: [Discuss-gnuradio] Include guard bug

2014-02-21 Thread Marcus Müller
Well don't worry, it wasn't actually causing problems ;) After fixing, I got out my zsh-foo and tried echo $(( $(git ls-files |grep .h$|xargs git grep --heading #ifndef INCLUDED|wc -l) - $(git ls-files |grep .h$|xargs git grep --heading #ifndef INCLUDED| uniq | wc -l) )) luckily, there are no

Re: [Discuss-gnuradio] Include guard bug

2014-02-21 Thread Marcus Müller
Hi Ruecan, this just the question if we should move away from #ifndef INCLUDED_FILENAME_OF_HEADER_H #define INCLUDED_FILENAME_OF_HEADER_H actual header content #endif to the more sanity-ensuring #pragma once preprocessor directive. Sadly, this was not related to your problem... Still have

Re: [Discuss-gnuradio] Include guard bug

2014-02-21 Thread Ruecan
Do you think that if I go back and try to install GR 3.7.0 instead, it may work ? -- View this message in context: http://gnuradio.4.n7.nabble.com/Issue-while-installing-GR-3-7-tp46435p46466.html Sent from the GnuRadio mailing list archive at Nabble.com.