Re: [Discuss-gnuradio] cmake build

[Josh Blum]

> I like it!
> ...some more verbose comments:
> 
> * All of my machines are pretty standard (all Ubuntu, 32 and 64 bit
>   flavours, versions 11.04 and 10.10). It works on both (with some minor
>   issues).
> 
> * One of the machines had an old UHD version, which manages to crash the
>   build:
> <snip>
> [ 88%] Building CXX object
> gr-uhd/lib/CMakeFiles/gnuradio-uhd.dir/gr_uhd_usrp_source.cc.o
> In file included from
> /home/braun/src/gnuradio-cmake/gr-uhd/lib/gr_uhd_usrp_source.cc:22:
> /home/braun/src/gnuradio-cmake/gr-uhd/include/gr_uhd_usrp_source.h:27:
> fatal error: uhd/usrp/multi_usrp.hpp: No such file or directory
> compilation terminated.
> make[2]: ***
> [gr-uhd/lib/CMakeFiles/gnuradio-uhd.dir/gr_uhd_usrp_source.cc.o] Error 1
> make[1]: *** [gr-uhd/lib/CMakeFiles/gnuradio-uhd.dir/all] Error 2
> </snip>
>   I believe this should be caught by the configuration process, not
>   during the build (...?)
> 

Does uhd/usrp/multi_usrp.hpp actually exist on the machine? And did
gnuradio (current master) build w/ this version of uhd? There may have
been an issue w/ an older uhd.pc file.

> * I didn't time it, but is the build time much faster? Even if this is
>   only subjective, it's an improvement.
> 

Ive noticed this too. :-)

> * The way the CMakeLists.txt look, this system seems much easier to
>   configure. Thumbs up! (Even though it's actually longer :)
> 

The cause of this may be that the job performed by makefile.swig.gen*,
runtests.in*, *.m4, *.am files is actually combined into the cmakelists.
Same functionality, less files.

> * One of the tests failed:
> <snip>
> 99% tests passed, 1 tests failed out of 83
> 
> Total Test time (real) =  71.21 sec
> 
> The following tests FAILED:
>          69 - qa_costas_loop_cc (Failed)
> Errors while running CTest
> make: *** [test] Error 8
> </snip>
> 

This is a known issue w/ gr-digital. Tom has a fix in his digital branch

> * I noticed radar-mono and sounder are missing. If that was done on
>   purpose, then good riddance. While they're interesting demos, they're
>   not really useful and I always disabled them.
> 

I believe those components will be removed in the near future.

Thanks for testing!

-josh