Bugs item #1637820, was opened at 2007-01-17 16:47
Message generated for change (Comment added) made by stmane
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=482468&aid=1637820&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: Fixed
Priority: 5
Private: No
Submitted By: Stefan Manegold (stmane)
Assigned to: Romulo Goncalves (romulog)
Summary: SQL fails to compile in case MonetDB4 is available

Initial Comment:
after recent changes, SQL fails to compile in case MonetDB4 is available:

 gcc -m64 -DHAVE_CONFIG_H -I. 
-I/ufs/manegold/_/scratch0/Monet/Testing/Current/sql/src/backends/monet4 
-I../../.. 
-I/ufs/manegold/_/scratch0/Monet/Testing/Current/sql/src/backends/monet4 
-I../../include 
-I/ufs/manegold/_/scratch0/Monet/Testing/Current/sql/src/backends/monet4/../../include
 -I../../common 
-I/ufs/manegold/_/scratch0/Monet/Testing/Current/sql/src/backends/monet4/../../common
 -I../../storage 
-I/ufs/manegold/_/scratch0/Monet/Testing/Current/sql/src/backends/monet4/../../storage
 -I../../server 
-I/ufs/manegold/_/scratch0/Monet/Testing/Current/sql/src/backends/monet4/../../server
 
-I/net/corona.ins.cwi.nl/export/scratch0/manegold/Monet/Testing/Current/.GNU.64.64.d.--disable-optimize_--enable-debug_--enable-assert.PREFIX./MONETDB/include/MonetDB
 
-I/net/corona.ins.cwi.nl/export/scratch0/manegold/Monet/Testing/Current/.GNU.64.64.d.--disable-optimize_--enable-debug_--enable-assert.PREFIX./MONETDB/include/MonetDB/common
 -I/net/corona.ins.cwi.nl/export/scratch0/manegold/Mone
 
t/Testing/Current/.GNU.64.64.d.--disable-optimize_--enable-debug_--enable-assert.PREFIX./MONETDB/include/MonetDB/gdk
 
-I/net/corona.ins.cwi.nl/export/scratch0/manegold/Monet/Testing/Current/.GNU.64.64.d.--disable-optimize_--enable-debug_--enable-assert.PREFIX./MONETDB4/include/MonetDB4
 
-I/net/corona.ins.cwi.nl/export/scratch0/manegold/Monet/Testing/Current/.GNU.64.64.d.--disable-optimize_--enable-debug_--enable-assert.PREFIX./MONETDB4/include/MonetDB4/monet
 
-I/net/corona.ins.cwi.nl/export/scratch0/manegold/Monet/Testing/Current/.GNU.64.64.d.--disable-optimize_--enable-debug_--enable-assert.PREFIX./MONETDB4/include/MonetDB4/plain
 
-I/net/corona.ins.cwi.nl/export/scratch0/manegold/Monet/Testing/Current/.GNU.64.64.d.--disable-optimize_--enable-debug_--enable-assert.PREFIX./MONETDB4/include/MonetDB4/contrib
 
-I/net/corona.ins.cwi.nl/export/scratch0/manegold/Monet/Testing/Current/.GNU.64.64.d.--disable-optimize_--enable-debug_--enable-assert.PREFIX./MONETDB4/include/MonetDB4/mapi
 -I
 
/net/corona.ins.cwi.nl/export/scratch0/manegold/Monet/Testing/Current/.GNU.64.64.d.--disable-optimize_--enable-debug_--enable-assert.PREFIX./MONETDB4/include/MonetDB4/calibrator
 -DLIBSQL_SERVER -Wall -W -std=c99 -g -Werror-implicit-function-declaration 
-Werror -Wno-unused-function -Wno-unused-label -c sql_gencode.c  -fPIC -DPIC -o 
.libs/lib_sql_server_la-sql_gencode.o
cc1: warnings being treated as errors
/ufs/manegold/_/scratch0/Monet/Testing/Current/sql/src/backends/monet4/sql_gencode.mx:
 In function 'backend_dumpstmt_':
/ufs/manegold/_/scratch0/Monet/Testing/Current/sql/src/backends/monet4/sql_gencode.mx:346:
 warning: enumeration value 'st_connection' not handled in switch
make[7]: *** [lib_sql_server_la-sql_gencode.lo] Error 1



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

>Comment By: Stefan Manegold (stmane)
Date: 2007-01-17 23:09

Message:
Logged In: YES 
user_id=572415
Originator: YES

The problem was (presumably) fixed; hence, we say so.
;-)


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

Comment By: Romulo Goncalves (romulog)
Date: 2007-01-17 22:10

Message:
Logged In: YES 
user_id=1498628
Originator: NO

This problem was resolved and also checked in.

Regards,
Romulo

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

Comment By: Stefan Manegold (stmane)
Date: 2007-01-17 17:57

Message:
Logged In: YES 
user_id=572415
Originator: YES

As long as SQL accepts M4, it should also compile with M4; otherwise,
configure should already tell that M4 is not supported any longer, or
simply ignore M4, in case M5 is available.

But until then, please make sql compile also if M4 is available.
In case it is too complicated to back-port your new functionality to M4,
just make sure that it (a) compiles, and (b) gives a proper error messages
in case a user tries to use with M4 the new functionality that is only
supported with M5.

Thanks!


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

Comment By: Romulo Goncalves (romulog)
Date: 2007-01-17 16:55

Message:
Logged In: YES 
user_id=1498628
Originator: NO

Yes you are right.
In my pc I only have M5 and not M4. I had the idea that M4 was dead.
I will install M4 and correct the problem. 
Should I run also the Mtest for M4?

Regards,
Romulo

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

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

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Monetdb-bugs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/monetdb-bugs

Reply via email to