On Mon, Sep 23, 2013 at 11:49:02AM -0500, William Harrington wrote: > > On Sep 22, 2013, at 8:22 PM, Ken Moffat wrote: >
> >On Sun, Sep 22, 2013 at 03:23:13PM -0500, William Harrington wrote: > >> > >>Also, there is a problem with Bison 3.0 and the 24th test suite with > >>constructs. It's already been addressed at bison-bugs: > >> > >>http://lists.gnu.org/archive/html/bug-bison/2013-09/msg00008.html > >> > >>I've had it hit me while testing our current git ppc64 pure64 build. > >>I haven't had it occur with x86 or x86_64 pure64 > >> > >>Sincerely, > >> > >>William Harrington > > > >As I've just noted in re-introducing myself to the list, my builds > >are much closer to LFS than cross-lfs. I wasn't planning to comment > >on the issues in my own build until I had any results, but since you > >are now asking... I see you've already noted that bison problem, > >for the moment it might just be a testsuite issue (haven't looked at > >the code, but perhaps the old 'signed char' has caught someone out > >again). > > With the attached one-line patch it no longer hangs on my ppc (this isn't all the same packages/versions that you guys are using). Both the version I was asked to test, and 3.0 with this one-line change, complete but get 1 error, for test 383. Not sure what the correct git tree (or branch) is, but google found this as a coloured diff. ĸen -- das eine Mal als Tragödie, dieses Mal als Farce
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. found at http://git.savannah.gnu.org/cgit/bison.git/diff/src/location.c?id=4c4191cec2b30632a31993a6e46fb241ab9c3434&ignorews=1 copied and pasted. --- bison-3.0/src/location.c.orig 2013-10-16 18:02:56.880658070 +0100 +++ bison-3.0/src/location.c 2013-10-16 18:04:12.676299868 +0100 @@ -188,7 +188,7 @@ /* Read the actual line. Don't update the offset, so that we keep a pointer to the start of the line. */ { - char c = getc (caret_info.source); + int c = getc (caret_info.source); if (c != EOF) { /* Quote the file, indent by a single column. */
_______________________________________________ Clfs-dev mailing list [email protected] http://lists.cross-lfs.org/listinfo.cgi/clfs-dev-cross-lfs.org
