On Fri, Oct 18, 2013 at 04:28:56PM +0200, Akim Demaille wrote: > hi Ken, > > Le 16 oct. 2013 à 20:57, Ken Moffat <[email protected]> a écrit : > > > For me (linux ppc) it allows the testsuite to complete. > > Great news! > > > There is now one failure: > > ERROR: 409 tests were run, > > 1 failed unexpectedly. > > 23 tests were skipped. > > > > which is > > > > 383: Exception safety api.value.type=variant FAILED > > (c++.at:912) > > Less great news :) > > > I looked for that commit in git, but I can't see it. > > I don't understand what you mean here. Are you looking for > the commit that introduced the test? The "variant" support? > No, the commit which fixed the test 24 hang - commit 4c4191cec2b30632a31993a6e46fb241ab9c3434 Author: Andreas Schwab <[email protected]> Date: Sat Sep 28 00:00:34 2013 +0200 location: fix EOF check * location.c (location_caret): Use int, not char, for values * from getc.
> > Google found > > it as a coloured diff, I couldn't see how to get a raw one, so I > > applied the one-line change by hand to the 3.0 release. > > What one-line change are you referring to? > As above - the change from char to int. I wasn't sure what else had changed, I just wanted to confirm that fixing location.c wasn't responsible for the failure in 383. > > There, the testsuite completed with the same error so the commit > > has not introduced a new failure. Things are moving forward. > > > > I'm attaching a tarball of everything from tests/testsuite.dir/383 > > in 3.0.20-e4678 except for the 'input' binary. > > Thanks a lot. I think I found the problem (the test suite is buggy, > but it most cases it will work by accident). I'm sad that I did not > get a type error in the test suite, I should have a look at how I > can get one. > > Could you please confirm that the bug is fixed with this tarball? > > Thanks! > > http://www.lrde.epita.fr/~akim/download/bison-3.0.22-fc8c9.tar.gz > http://www.lrde.epita.fr/~akim/download/bison-3.0.22-fc8c9.tar.xz ## ------------- ## ## Test results. ## ## ------------- ## 410 tests were successful. 23 tests were skipped. The skipped tests are for Doxygen and java, neither of which are installed. Thanks for fixing it. ĸen -- das eine Mal als Tragödie, dieses Mal als Farce
