Bug 4048 - Patch for Solaris (i86pc and SPARC)
Status: RESOLVED FIXED
Alias: None
Product: ioquake3
Classification: Unclassified
Component: Platform
Version: unspecified
Hardware: Sun Solaris
: P2 normal
Assignee: Zachary J. Slater
QA Contact: ioquake3 bugzilla mailing list
URL:
Depends on:
Blocks:
 
Reported: 2009-04-27 11:30 EDT by Vincent Cojot
Modified: 2009-05-03 14:51:21 EDT
1 user (show)

See Also:


Attachments
Diff against SVN_HEAD (8.16 KB, patch)
2009-04-27 11:32 EDT, Vincent Cojot

Description Vincent Cojot 2009-04-27 11:30:58 EDT
Hello,

This patch adds a few new files under trunk/misc/setup/pkg and changes trunk/Makefile so that:
1) libSDL can now be linked static on Solaris (Solaris 10 provides SDL 1.2.5 and I had a few users report issues with the older system-wide SDL).
2) compile vm_sparc for Solaris/sparc as well.
3) split the iquake3d package: idquake3d (patch data files) and idquake3m (demo data file).

Please apply if you have time,

Thanks.

Vincent
Comment 1 Vincent Cojot 2009-04-27 11:32:40 EDT
Created attachment 2043 [details]
Diff against SVN_HEAD
Comment 2 Ludwig Nussel 2009-05-02 13:14:17 EDT
(In reply to comment #0)
> 1) libSDL can now be linked static on Solaris (Solaris 10 provides SDL 1.2.5
> and I had a few users report issues with the older system-wide SDL).

We don't link SDL statically on any platform so please don't do that on Solaris either.

Also your versioning scheme ($(VERSION)-rSVNxxx) doesn't match the one we use ($(VERSION)_SVNxxx), please use the opportunity to change that unless there's a good reason to do it differntly.
Comment 3 Vincent Cojot 2009-05-03 09:36:48 EDT
Hi Ludwig,
Thanks for the answer.
I'll be providing a dynamic libSDL.so on sparc and x86 under /usr/local/share/games/quake3 *COPYDIR for Solaris*.
I changed my pach to adapt the versioning scheme to your standards.
Here's the new patch.
Comment 4 Ludwig Nussel 2009-05-03 14:51:21 EDT
You attached it to the wrong bug. Anyways, applied with modifications. Instead of setting rpath (I suppose that's what -R does) I'd set LD_LIBRARY_PATH in the start script like on Linux. Why don't you use the Linux start script anyways?