Post Reply 
 
Thread Rating:
  • 0 Votes - 0 Average
  • 1
  • 2
  • 3
  • 4
  • 5
OS X Builds again?
12-13-2008, 09:30 AM
Post: #1
OS X Builds again?
Have we had any successful OS X builds since refactor? I gave it a run this morning because I had a few minutes (where does my time go?) and it fails pretty horribly. I'm guessing there have been no updates to the OS X project files.

_____
Cotharyus
Remember: Horsepower is how hard you hit the wall. Torque is how far you take it with you.
http://cotharyus.net
Visit this user's website Find all posts by this user
Quote this message in a reply
12-14-2008, 01:20 AM
Post: #2
 
I think you are correct. As far as I know it *should* compile fine, though, once the project files are set up correctly....
Find all posts by this user
Quote this message in a reply
12-16-2008, 12:44 PM
Post: #3
 
Yeah, the project file is the biggy though. I've messed with minor things before, but this looks like it might require a complete redo. I'm picking at it, but I don't have it right yet.

_____
Cotharyus
Remember: Horsepower is how hard you hit the wall. Torque is how far you take it with you.
http://cotharyus.net
Visit this user's website Find all posts by this user
Quote this message in a reply
12-16-2008, 10:38 PM
Post: #4
 
OK, let me know if you run into any compile problems.
Find all posts by this user
Quote this message in a reply
12-18-2008, 08:44 AM
Post: #5
 
ok r2261 should at least compile, didn't have luck running it yet though.

but you need to expand bullet-2.73-sp1.tgz, i don't know why it is stored only compressed in svn.

btw. why was the PCH stuff removed? i had to add ifdefs for includes all over the place again ... ;-(

bye, julian
Find all posts by this user
Quote this message in a reply
12-18-2008, 12:30 PM
Post: #6
 
Latest SVN (revision 2262) doesn't compile here (Linux).

Code:
src/game.cpp: In member function ‘void GAME::InitializeThreading()’:
src/game.cpp:150: error: ‘info’ was not declared in this scope
src/game.cpp:151: error: ‘info’ was not declared in this scope
scons: *** [build/game.o] Error 1

Free Gamer - open source games lists and commentary
http://freegamer.blogspot.com
Visit this user's website Find all posts by this user
Quote this message in a reply
12-18-2008, 05:25 PM
Post: #7
 
R2263 is fixed for unix. Julian accidentally deleted a line when he replaced the #ifdef block -- no biggie, easy fix.
Find all posts by this user
Quote this message in a reply
12-18-2008, 05:28 PM
Post: #8
 
abs1nth Wrote:why was the PCH stuff removed?

Ignorance on my part of what the pch stuff was for when I did the refactor. Sorry about that. :oops:
Find all posts by this user
Quote this message in a reply
12-20-2008, 08:45 AM
Post: #9
 
here is a patch that re-adds the pch stuff:
In order to view links, you must have to reply to this thread.


probably needs some adjusting on win/lin and either some scons magic to include pch.h in every compile or add a include pch.h to every file. don't remember how that was done previously.
Find all posts by this user
Quote this message in a reply
12-20-2008, 02:38 PM
Post: #10
 
abs1nth Wrote:either some scons magic to include pch.h in every compile or add a include pch.h to every file. don't remember how that was done previously.

Previously there was an #include "pch.h" in every file. It seems kind of messy to do it that way, but I can't think of any problem with it besides just obscuring exactly what external library includes your file is depending on.

But, prior to the patch you already fixed the includes for everything to be correct on OSX? If so, then maybe it's not worth the effort to add #include pch.h everywhere; I don't expect any new external dependencies. Thoughts?
Find all posts by this user
Quote this message in a reply
12-21-2008, 09:59 AM
Post: #11
 
joevenzon Wrote:If so, then maybe it's not worth the effort to add #include pch.h everywhere; Thoughts?

well, of course all other includes are removed, so it is a net-win (LoC wise) ;-)

also if the pch.h is actually precompiled there is a significant compile-time speedup - but of course there is the issue of doing this with scons. a quick search yields this:
In order to view links, you must have to reply to this thread.
Find all posts by this user
Quote this message in a reply
Post Reply 


Forum Jump:


User(s) browsing this thread: 1 Guest(s)