Summary: | Xorg CVS head won't build on Solaris/sparc | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | xorg | Reporter: | Alan Coopersmith <alan.coopersmith> | ||||||
Component: | Server/Ext/GLX | Assignee: | Alan Coopersmith <alan.coopersmith> | ||||||
Status: | RESOLVED FIXED | QA Contact: | |||||||
Severity: | blocker | ||||||||
Priority: | high | CC: | ajax, kem | ||||||
Version: | git | ||||||||
Hardware: | SPARC | ||||||||
OS: | Solaris | ||||||||
Whiteboard: | |||||||||
i915 platform: | i915 features: | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 351 | ||||||||
Attachments: |
|
Description
Alan Coopersmith
2004-08-16 16:20:26 UTC
Created attachment 654 [details] [review] Patch to fix build on Solaris/SPARC Add LargePICTable/PICFLAGS to GL Imakefiles as necessary Add include paths to xc/lib/GL/mesa/src/SPARC/Imakefile that match the similar code in xc/lib/GL/mesa/src/x86 Took a look at your patch, and the issues that I'm concerned about are: - #define LargePICTable YES shouldn't have any effect in the server-side Imakefiles since that define is only used in Library.tmpl, so are those defines really needed? - Are these changes needed by only Solaris? Will they cause any problems with Linux on SPARC? I included the defines out of habit - they probably aren't needed. As for Linux on SPARC, I have no idea. I have no Linux/SPARC machines around to test with, nor do I know anyone who does. It looks like it does not define LargePositionIndependentCFlags but simply always uses the large PIC table flags. I can change the patch to be '#if defined(SunArchitecture) && defined(SparcArchitecture) && defined(LargePositionIndependentCFlags)' to prevent issues there. (In reply to comment #3) > As for Linux on SPARC, I have no idea. I have no Linux/SPARC machines around > to test with, nor do I know anyone who does. It looks like it does not define > LargePositionIndependentCFlags but simply always uses the large PIC table flags. > I can change the patch to be '#if defined(SunArchitecture) && > defined(SparcArchitecture) && defined(LargePositionIndependentCFlags)' to prevent > issues there. That would probably be safer to do. After you've updated the patch and tested it out, please let me know and I'll check in the changes. Created attachment 678 [details] [review] Updated patch Updated patch as discussed with Kevin Tested on Solaris sparc with both Sun cc and gcc Patch checked in. Closing. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.