On Wed, Oct 26, 2011 at 7:49 AM, Paul M. Bendixen <paulbendi...@gmail.com>wrote:

> Did a little more digging
> cd [gitdir]/grc/
> find . -name "*" | xargs grep digital
>
> Returns nothing, on the other hand grepping blks2.gmsk_mod returns:
> ./Makefile.am: blks2_gmsk_mod.xml \
> ./block_tree.xml: <block>blks2_gmsk_mod</block>
> ./blks2_gmsk_mod.xml: <key>blks2_gmsk_mod</key>
> ./blks2_gmsk_mod.xml: <make>blks2.gmsk_mod(
>


Fixing these now. About to push the patch.



> In other words: the XML structure of the grc that has to do with anything
> moved to digital must be updated, before 3.5 can be compared to a stable
> environment.
>
> XML has never been my strong suit, so I won't start looking into it.
> Furthermore some of the blocks have changed input parameters (e.g.
> gmsk_demod).
>
> Perhaps merging next into master was a little premature?
>
> Best regards
> Paul
>

I don't think so at all. The problem is there are no automated test for this
kind of stuff, and there's a lot of code that's of concern here. The only
way we could get these kinds of error reports was to get the real users of
GNU Radio to test them. These problems have been around in the old 'next'
branch for a while, but without being used, there was no way for us to get
these kinds of bug reports that is allowing us to fix them.

In other words, I really appreciate you checking these things out and
reporting the errors. This is also why we have stable releases. For people
who have environments where stability is a concern, they should stick with
the releases.

I went through and checked the GRC blocks against everything I moved into
gr-digital, so hopefully there is nothing left that I missed. I wonder if we
can make a piece of automatic test code that would try to load each of the
GRC XML blocks and report an error if it can't be loaded properly?

Thanks! Let me know if you find anything else.

Tom
_______________________________________________
Discuss-gnuradio mailing list
Discuss-gnuradio@gnu.org
https://lists.gnu.org/mailman/listinfo/discuss-gnuradio

Reply via email to