This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] intl: Merge with gettext version 0.19.3


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.

--
Chris Metcalf, EZChip Semiconductor
http://www.ezchip.com


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]