Re: "make gas-test" broken?

[Dave Korn <dave dot korn dot cygwin at googlemail dot com>]

Doug Evans wrote:

> Ya, that's the problem I was referring to in
> http://sourceware.org/ml/cgen/2009-q3/msg00056.html.
> I've changed method test-data of class <keyword> to flag a warning and
> compensate.  It's not ideal, technically wrong, but for right now I
> don't mind in this particular case.

  Ah, gotcha; that's this bit, right?

> +	    ;; If an ifield has, e.g., 2 bits (values 0,1,2,3) and the keyword
> +	    ;; only has two values, e.g. (foo 0) (bar 1), then we can get
> +	    ;; invalid requests, i.e. for ifield values of 2 and 3.
> +	    ;; It's not clear what to do here, but it seems like this is an
> +	    ;; error in the description file.

  I also have some ifields like that, they're part of the opcode selection and
not all combinations are valid.  (I'll test filling them out with dummy
entries before I update my sandbox just to see if that would have made a
difference.)

> Ya, this is a change in Guile's behaviour (circa 1.4?).  string-append
> use to accept symbols, it no longer does.

  I was just about to ask about guile versions.  I've been noticing testsuite
FAILs which could be related - I'll take a closer look at any that remain
after I've updated.

> Most of cgen has been updated, alas gas-test.scm wasn't - I've fixed it,
> give it another try.

  Thank you Doug!  I'll be back in touch.

    cheers,
      DaveK