Re: tinderboxes don't need to upgrade bison anymore

On Wed, Jul 10, 2013 at 11:24 AM, Lionel Elie Mamane <lionel@mamane.lu> wrote:
> On Wed, Jul 10, 2013 at 04:32:33PM +0200, Lionel Elie Mamane wrote:
> > On Wed, Jul 10, 2013 at 02:27:51PM +0300, Tor Lillqvist wrote:
>
> > > OS X 10.8 has Bison 2.3 (...) We shouldn't be requiring anything
> > > newer, (...) (Sure, we already have broken that promise a bit as we
> > > require automake and autoconf, but do we *really* need any newer
> > > features of Bison?)
>
> > Apparently, the way it is implemented in Bison 2.3 interacts badly
> > with preprocessor tricks done in sqlbison.y, while it works well for
> > the implementation in Bison 2.4 and 2.5.
>
> Concretely, the overall problem is that Bison 2.3 puts much (more) of
> sqlbison.y in sqlbison.hxx, which is #included in sqlflex.l. This
> makes conflicts / pollutes the namespace. There were two compile-time
> errors:
>
> 1) aEmptyString defined twice. I renamed the one of sqlflex.l

Actually you can s/aEmptyString/OUString()/g in both files and delete
aEmptyString altogether

Norbert