On 11 December 2014 at 16:37, Chris Metcalf <[email protected]> wrote: > On 12/10/2014 10:15 AM, Will Newton wrote: >> >> This patch merges the latest release of gettext into the intl >> subdirectory. The initial motivation was to include the plural.y >> changes which enable building with bison 3.0, but the majority >> of the other changes are merely cosmetic so it seemed like merging >> the whole directory was simpler than trying to take it piecemeal. > > > This appears to break setups like RHEL 6 where we still use bison 2.4.1: > > /usr/bin/bison --yacc --name-prefix=__gettext --output plural.c plural.y > plural.y:46.18-21: syntax error, unexpected identifier > > We don't seem to document or test for any particular version of bison, > though I > see the shipped plural.c file is still from bison 2.7. > > It would be nice if this actually continued to work with the older bison. > I just changed that line to "%define api.pure", which works, with some > -Wundef's, > but honestly I have no idea what I'm doing. :-) Equivalently I can just > touch > the shipped plural.c to avoid running bison; again we do get -Wundef's, > though.
Ugh, I only tested with 2.7 and 3.0. Which Wundefs happen if you touch plural.c? -- Will Newton Toolchain Working Group, Linaro
