On Mon, Oct 09, 2006 at 09:27:59AM -0700, Paul Eggert wrote: > Bob Rossi <[EMAIL PROTECTED]> writes: > > > That's odd, the test suite passes for me. I just did an update, and > > reran the tests, and they still pass. I'm posting the patch again in > > case I messed something up the first time. > > No, the patch is right: it matches what I have. > 90: Calculator %push-parser %locations FAILED (calc.at:574) > 96: Calculator %push-parser %error-verbose %debug %locations %defines > %name-prefix=calc %verbose %yacc FAILED (calc.at:583)
OK, I understand now. It's because you have to 'cp push.c yacc.c' in order for the new tests to pass. They will not work with yacc.c because it doesn't support the push parser. Should I modify the test suite somehow so that tests 90 and 96 explicitly use the push.c skeleton? Thanks, Bob Rossi
