Bug 3426 - Fix compile time errors for SGI IRIX
Status: RESOLVED FIXED
Alias: None
Product: ioquake3
Classification: Unclassified
Component: Platform
Version: GIT MASTER
Hardware: SGI IRIX
: P3 blocker
Assignee: Zachary J. Slater
QA Contact: ioquake3 bugzilla mailing list
URL:
Depends on:
Blocks:
 
Reported: 2007-11-14 23:23 EST by Patrick Baggett
Modified: 2007-11-26 15:26:26 EST
0 users

See Also:


Attachments
Patch against SVN rev 1212 for Quake3 to build on IRIX 6.5 (2.89 KB, patch)
2007-11-14 23:25 EST, Patrick Baggett

Description Patrick Baggett 2007-11-14 23:23:09 EST
IOQuake3 does not build on SGI IRIX 6.5

There was a patch on Nekochan.net (an SGI/IRIX fan site) that patched IOQuake3 to build on IRIX successfully, and generally stop bitrot. The patch is originally described at: http://forums.nekochan.net/viewtopic.php?t=12556

This is for an older version of IOQuake3, and so I manually applied fixes to the SVN tree rev 1212, which is the newest at the time of this writing.

I have successfully compiled this on IRIX, and even played a few games online with some guys. As mentioned in the forum link above, using bots causes the game to crash. That is a different bug, as this report is for making it build. I do not have a sound card in this machine, so I cannot say anything about the sound quality.

Most of the groundwork for this was not done by me but by "Mare" of Nekochan. It looks like the patch did not make it into the SVN tree.
Comment 1 Patrick Baggett 2007-11-14 23:25:25 EST
Created attachment 1578 [details]
Patch against SVN rev 1212 for Quake3 to build on IRIX 6.5

Compiled and tested on IRIX 6.5.28 with MIPSpro 7.4.1m, on SGI Fuel with V12 with  MIPS R14000 processor.
Comment 2 Tim Angus 2007-11-19 19:15:45 EST
Could you try using time.h instead of sys/time.h? The latter isn't available on Windows. I know you're guarding it but we're trying to keep platform #ifdeffery to an absolute minimum. If time.h could be included without the guard that would be better.
Comment 3 Tim Angus 2007-11-24 13:49:31 EST
Did you get around to trying the alternative header?
Comment 4 Tim Angus 2007-11-26 15:26:26 EST
Actually this header is only included in unixy places, so it's fine to use sys/time.h in these places, without a guard. Committed in r1218, thanks.