Bugs item #2872565, was opened at 2009-10-04 15:49
Message generated for change (Settings changed) made by sjoerd
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=482468&aid=2872565&group_id=56967

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Build
Group: SQL CVS Head
>Status: Closed
>Resolution: Wont Fix
Priority: 5
Private: No
Submitted By: Mitja Kleider (mkleider)
Assigned to: Sjoerd Mullender (sjoerd)
Summary: building sql with bison++ 1.21.11 fails

Initial Comment:
building sql with bison++ fails because of warnings/errors in bison++, see 
attached output of `make`

System information:

Ubuntu 9.04 Jaunty Jackalope

gcc 4.3.3-5ubuntu4
flex-old 2.5.4a-7build1
bison++ 1.21.11-3

autoconf 2.63-2ubuntu1
automake 1:1.10.2-0ubuntu3
libtool 2.2.6a-1ubuntu1

----------------------------------------------------------------------

>Comment By: Sjoerd Mullender (sjoerd)
Date: 2009-10-05 18:15

Message:
By adding some defines to sql_parser.mx I can get this through bison++,
except for one issue.  The issue is a comparison between int and size_t
(the result of sizeof).  This comparison is in the generated code (i.e. not
in our code), so this is not something we can fix.  Since we do not support
bison++, I am not willing to override the normal treat warnings as errors
option that we give to the compiler.  If you want to use bison++ you can
override that option for this file.

I am closing this with a Wont Fix since the remaining problem is not in
our code.

----------------------------------------------------------------------

Comment By: Sjoerd Mullender (sjoerd)
Date: 2009-10-05 14:33

Message:
I closed this bug prematurely.
The error I got while compiling MonetDB4 using bison++ was fixed, but not
the one that caused sql to not compile.

----------------------------------------------------------------------

Comment By: Sjoerd Mullender (sjoerd)
Date: 2009-10-05 14:07

Message:
Fixed in CVS by making sure the generated *.tab.c file starts with
including our config.h file.
No test needed.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=482468&aid=2872565&group_id=56967

------------------------------------------------------------------------------
Come build with us! The BlackBerry® Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay 
ahead of the curve. Join us from November 9-12, 2009. Register now!
http://p.sf.net/sfu/devconf
_______________________________________________
Monetdb-bugs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/monetdb-bugs

Reply via email to