[Patch-gnuradio] Bug in GRC, bugs in grc-examples and unit tests [Was: GRC: Just compile XML, no GUI]

[Martin Braun]

On Thu, Feb 16, 2012 at 02:28:21PM -0500, Paul Miller wrote:
> On Thu, Feb 16, 2012 at 10:19:31AM -0800, Josh Blum wrote:
> > > I was wondering if it's a big deal to give gnuradio-companion a switch
> > > to just compile a .grc file without firing up the GUI.
> 
> > Give this a shot: https://github.com/jettero/grcc/blob/grcc/grcc
> > -Josh
> 
> This is exactly why I wrote grcc.  It should probably be a switch
> in the gnuradio-companion script, but grcc does the job for now.
> I'm open to forks and patches, I really just cobbled together
> something that works for me, suggestions welcome.

Yep, that's exactly what I wanted. I think this should be part of grc--I
posted a patch here: https://github.com/mbant/gnuradio/tree/grcunittest

This contains two commits:
76e918438e5 simply adds grcc to gnuradio-companion (plus one more switch
to set the dir). This is pretty simple, and I'd suggest it for 'next'.

724a3ff8798d adds a unit test to gr-howto-write-a-block for the grc
files. This one's very hackish, it shouldn't be merged into anything,
but I'd like to suggest something like this for GR. The current next
branch has some broken grc examples, and with something like this it
would be super easy to track the working state of these grc files.

The most relevant change is in
https://github.com/mbant/gnuradio/blob/grcunittest/gr-howto-write-a-block/apps/CMakeLists.txt
(you can see my solution is ugly, non-portable and so on but does the
trick).

It also has a workaround for a bug
(http://gnuradio.org/redmine/issues/485) in GRC. Probably should have
been two commits.

MB

-- 
Karlsruhe Institute of Technology (KIT)
Communications Engineering Lab (CEL)

Dipl.-Ing. Martin Braun
Research Associate

Kaiserstraße 12
Building 05.01
76131 Karlsruhe

Phone: +49 721 608-43790
Fax: +49 721 608-46071
www.cel.kit.edu

KIT -- University of the State of Baden-Württemberg and
National Laboratory of the Helmholtz Association

pgp_1xC4dellF.pgp
Description: PGP signature