Thread: [Opentnl-general] Congrats on the new release!
Brought to you by:
mark_frohnmayer,
s_alanet
From: Brenden T. <br...@rc...> - 2004-04-21 16:01:13
|
Congratulations on the release of OpenTNL, now fix my bugs :-b Compiled with MinGW 3.3.3, MSYS umm, looks like 1.09 but I've got binutils 2.14 for it. ghostConnection.cpp|746| error: syntax error before `)' token || ghostConnection.cpp: In constructor ` || TNL::RPC_GhostConnection_rpcGhostAlwaysStarting::RPC_GhostConnection_rpcGhostAlwaysStarting() || ': ghostConnection.cpp|746| error: `mFuncPtr' undeclared (first use this function) ghostConnection.cpp|746| error: (Each undeclared identifier is reported only || once for each function it appears in.) || ghostConnection.cpp: At global scope: ghostConnection.cpp|759| error: syntax error before `)' token ghostConnection.cpp|772| error: syntax error before `)' token ghostConnection.cpp|796| error: syntax error before `)' token || make[1]: *** [ghostConnection.o] Error 1 || make[1]: Leaving directory `/d/Documents and Settings/Brenden/Desktop/Dev/Open TNL/src/opentnl-src-1.0/tnl' || make: *** [default] Error 2 |
From: Ben G. <be...@ga...> - 2004-04-21 17:12:47
|
Interesting. Do you know what version of GCC you're using? Ben Brenden T. wrote: > Congratulations on the release of OpenTNL, now fix my bugs :-b > > Compiled with MinGW 3.3.3, MSYS umm, looks like 1.09 but I've got > binutils 2.14 for it. > > ghostConnection.cpp|746| error: syntax error before `)' token > || ghostConnection.cpp: In constructor ` > || > TNL::RPC_GhostConnection_rpcGhostAlwaysStarting::RPC_GhostConnection_rpcGhostAlwaysStarting() > > || ': > ghostConnection.cpp|746| error: `mFuncPtr' undeclared (first use this > function) > ghostConnection.cpp|746| error: (Each undeclared identifier is > reported only > || once for each function it appears in.) > || ghostConnection.cpp: At global scope: > ghostConnection.cpp|759| error: syntax error before `)' token > ghostConnection.cpp|772| error: syntax error before `)' token > ghostConnection.cpp|796| error: syntax error before `)' token > || make[1]: *** [ghostConnection.o] Error 1 > || make[1]: Leaving directory `/d/Documents and > Settings/Brenden/Desktop/Dev/Open TNL/src/opentnl-src-1.0/tnl' > || make: *** [default] Error 2 > > > > ------------------------------------------------------- > This SF.Net email is sponsored by: IBM Linux Tutorials > Free Linux tutorial presented by Daniel Robbins, President and CEO of > GenToo technologies. Learn everything from fundamentals to system > administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click > _______________________________________________ > Opentnl-general mailing list > Ope...@li... > https://lists.sourceforge.net/lists/listinfo/opentnl-general > > |
From: Brenden T. <br...@rc...> - 2004-04-21 17:19:42
|
Ben Garney wrote: > Interesting. Do you know what version of GCC you're using? > > Ben > > Brenden T. wrote: > >> Congratulations on the release of OpenTNL, now fix my bugs :-b >> >> Compiled with MinGW 3.3.3, MSYS umm, looks like 1.09 but I've got >> binutils 2.14 for it. >> gcc 3.3.3 |
From: Mark F. <ma...@ga...> - 2004-04-21 17:26:29
|
Yeah, that error is why MinGW isn't on the supported compilers list for Win32 :) Strangely GCC doesn't give this error on either linux or OS X. Perhaps it's a difference in the cdecl syntax? - Mark Brenden T. wrote: > Congratulations on the release of OpenTNL, now fix my bugs :-b > > Compiled with MinGW 3.3.3, MSYS umm, looks like 1.09 but I've got > binutils 2.14 for it. > > ghostConnection.cpp|746| error: syntax error before `)' token > || ghostConnection.cpp: In constructor ` > || > TNL::RPC_GhostConnection_rpcGhostAlwaysStarting::RPC_GhostConnection_rpcGhostAlwaysStarting() > > || ': > ghostConnection.cpp|746| error: `mFuncPtr' undeclared (first use this > function) > ghostConnection.cpp|746| error: (Each undeclared identifier is > reported only > || once for each function it appears in.) > || ghostConnection.cpp: At global scope: > ghostConnection.cpp|759| error: syntax error before `)' token > ghostConnection.cpp|772| error: syntax error before `)' token > ghostConnection.cpp|796| error: syntax error before `)' token > || make[1]: *** [ghostConnection.o] Error 1 > || make[1]: Leaving directory `/d/Documents and > Settings/Brenden/Desktop/Dev/Open TNL/src/opentnl-src-1.0/tnl' > || make: *** [default] Error 2 > > > > ------------------------------------------------------- > This SF.Net email is sponsored by: IBM Linux Tutorials > Free Linux tutorial presented by Daniel Robbins, President and CEO of > GenToo technologies. Learn everything from fundamentals to system > administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click > _______________________________________________ > Opentnl-general mailing list > Ope...@li... > https://lists.sourceforge.net/lists/listinfo/opentnl-general |
From: Brenden T. <br...@rc...> - 2004-04-21 17:51:57
|
Mark Frohnmayer wrote: > Yeah, that error is why MinGW isn't on the supported compilers list > for Win32 :) > Whoah, seriously? I assumed that Torque did support MinGW. Mea culpa, sorry to bother you all. I'll switch to VC++. |
From: Mark F. <ma...@ga...> - 2004-04-21 17:57:11
|
We'd be happy to support MinGW - we just don't right now (since it doesn't compile the RPC stuff)... but then, this is an Open Source project... ;) - Mark Brenden T. wrote: > Mark Frohnmayer wrote: > >> Yeah, that error is why MinGW isn't on the supported compilers list >> for Win32 :) >> > Whoah, seriously? I assumed that Torque did support MinGW. Mea > culpa, sorry to bother you all. I'll switch to VC++. > > > > > ------------------------------------------------------- > This SF.Net email is sponsored by: IBM Linux Tutorials > Free Linux tutorial presented by Daniel Robbins, President and CEO of > GenToo technologies. Learn everything from fundamentals to system > administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click > _______________________________________________ > Opentnl-general mailing list > Ope...@li... > https://lists.sourceforge.net/lists/listinfo/opentnl-general |
From: Ben G. <be...@ga...> - 2004-04-21 17:58:19
|
But this is TNL, not Torque. ;) Brenden T. wrote: > > Whoah, seriously? I assumed that Torque did support MinGW. Mea > culpa, sorry to bother you all. I'll switch to VC++. > |
From: Stephane V. <sv...@fr...> - 2004-04-22 04:52:46
Attachments:
tnlTypes.h
|
Hi, > Strangely GCC doesn't give this error on either linux or OS X. Perhaps= =20 > it's a difference in the cdecl syntax? You got the point. In fact for both __CYGWIN__ and __MINGW32__ the FN_CDE= CL constant must be defined to nothing like under Linux for instance. There are also some missing constants under mingw32 so I added them direc= tly in tnlTypes.h in a test to check if __MINGW32__ is defined. Now I got everything to compile fine with standard gcc cygwin and with gcc -mno-cygwin. For pure cygwin, TNL_OS is defined to linux whereas for mingw it is defin= ed to win32. Of course, for mingw compile, I had to add "-mno-cygwin" to each Makefile= (or makefile in libtomcrypt) and I had to add -lwsock32 to the Makefiles in t= he master, masterclient and test subdirectories. No changes to Makefiles where required for cygwin, just the new tnlTypes.= h. Here you will find (attached) my tnlType.h file that I modified from yest= erday's CVS. Could any devel commit this patch if it proves itself to be ok ? I dunno how to integrate the mingw32 required changes to Makefiles though= .. I should say I haven't thought about it. Is anyone planning to use a conf= igure script ? Regards, Stephane |
From: Ben G. <be...@ga...> - 2004-04-23 01:49:55
|
Stephane, It looks like a good fix. Nice work! We'll have to run it past Quigley, our linux build master, so that he can approve it (since MingW/Cygwin both use the same toolchain as linux), so it may take a few days to get feedback. He's currently at the Linux Desktop Summit, and thus incommunicado. :) You will also need to fill out and send to us the form located at <http://www.opentnl.org/agreement.html>. It's a joint copyright assignment; an unfortunate necessity for projects operating under dual licenses. :) If you have any questions feel free to ask (though you might want to do it off-list). Regards, Ben Garney Stephane Vaxelaire wrote: >Hi, > > > >>Strangely GCC doesn't give this error on either linux or OS X. Perhaps >>it's a difference in the cdecl syntax? >> >> > >You got the point. In fact for both __CYGWIN__ and __MINGW32__ the FN_CDECL >constant must be defined to nothing like under Linux for instance. > >There are also some missing constants under mingw32 so I added them directly >in tnlTypes.h in a test to check if __MINGW32__ is defined. >Now I got everything to compile fine with standard gcc cygwin and with >gcc -mno-cygwin. >For pure cygwin, TNL_OS is defined to linux whereas for mingw it is defined to >win32. > >Of course, for mingw compile, I had to add "-mno-cygwin" to each Makefile (or >makefile in libtomcrypt) and I had to add -lwsock32 to the Makefiles in the >master, masterclient and test subdirectories. >No changes to Makefiles where required for cygwin, just the new tnlTypes.h. > >Here you will find (attached) my tnlType.h file that I modified from yesterday's >CVS. > >Could any devel commit this patch if it proves itself to be ok ? >I dunno how to integrate the mingw32 required changes to Makefiles though.. >I should say I haven't thought about it. Is anyone planning to use a configure >script ? > >Regards, > >Stephane > |
From: John Q. <jo...@lo...> - 2004-04-25 18:22:02
|
Your tnlTypes.h fixes look good. One question, are the _HEAP defines possibly in some mingw header file? If so, it would be better to include that header file rather then redefine them in TNL. As far as the Makefiles go, the ones in TNL are quick and dirty and need a lot of work. Aside from configuration, they don't properly detect changes in header files or dependent libraries. I'd love to see an automake solution implemented, but I don't have time to work on it myself at the moment. Would you like to? :) Without autoconf, you could use a scheme similar to torque for setting up the flags for different compilers. Basically the main makefile includes a top level "configuration" makefile that defines the CFLAGS and other settings specific to that compiler. So you would configuration files like conf.GCC.MINGW.mk conf.GCC.LINUX.mk And the appropriate one would be included by the top-level makefile. On Apr 21, 2004, at 9:45 PM, Stephane Vaxelaire wrote: > Hi, > >> Strangely GCC doesn't give this error on either linux or OS X. >> Perhaps >> it's a difference in the cdecl syntax? > > You got the point. In fact for both __CYGWIN__ and __MINGW32__ the > FN_CDECL > constant must be defined to nothing like under Linux for instance. > > There are also some missing constants under mingw32 so I added them > directly > in tnlTypes.h in a test to check if __MINGW32__ is defined. > Now I got everything to compile fine with standard gcc cygwin and with > gcc -mno-cygwin. > For pure cygwin, TNL_OS is defined to linux whereas for mingw it is > defined to > win32. > > Of course, for mingw compile, I had to add "-mno-cygwin" to each > Makefile (or > makefile in libtomcrypt) and I had to add -lwsock32 to the Makefiles > in the > master, masterclient and test subdirectories. > No changes to Makefiles where required for cygwin, just the new > tnlTypes.h. > > Here you will find (attached) my tnlType.h file that I modified from > yesterday's > CVS. > > Could any devel commit this patch if it proves itself to be ok ? > I dunno how to integrate the mingw32 required changes to Makefiles > though.. > I should say I haven't thought about it. Is anyone planning to use a > configure > script ? > > Regards, > > Stephane<tnlTypes.h> |
From: Stephane V. <sv...@fr...> - 2004-04-26 04:17:00
|
Hi, > Your tnlTypes.h fixes look good. One question, are the _HEAP defines=20 > possibly in some mingw header file? If so, it would be better to=20 > include that header file rather then redefine them in TNL. I agree with that, but I haven't found them in any mingw headers. Only in VC++ ones. > detect changes in header files or dependent libraries. I'd love to see= =20 > an automake solution implemented, but I don't have time to work on it=20 > myself at the moment. Would you like to? :) Sure, I may give it a try I have some time I could use for that. Stephane |
From: Stephane V. <sv...@fr...> - 2004-04-26 04:38:52
|
I had troubles compiling zap under mingw. I had to change the (ALCint*)contextData into (ALint*)contextData in sfx.cpp:218. The game seems to work so I think that could be a fix because not all ope= nal versions define ALCint, but ALint always is. It worked fine under mingw, linux and MacOS X. Stephane |
From: Stephane V. <sv...@fr...> - 2004-04-26 13:39:09
Attachments:
configure.in
|
Here is an updated configure.in that won't stop if it doesn't detect Open= GL. |