Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 61103 - glheader.patch in nvidia-glx-1.0.6111 breaks glext apps
Summary: glheader.patch in nvidia-glx-1.0.6111 breaks glext apps
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Library (show other bugs)
Hardware: All All
: High normal (vote)
Assignee: Gentoo X packagers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-08-20 20:43 UTC by SpanKY
Modified: 2004-09-09 17:17 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description SpanKY gentoo-dev 2004-08-20 20:43:08 UTC
forcing inclusion of glext.h and the use of glext prototypes causes programs (like glaxium) to fail building because they use the opposite code path in glext.h (when GL_GLEXT_PROTOTYPES is not defined)
Comment 1 Andrew Bevitt 2004-08-27 10:10:15 UTC
Would this occure still IF we removed the checks for GL_GLEXT_PROTOTYPES and just had everything defined from both code paths? Basically is the failure a duplicate definition or something else?
Comment 2 SpanKY gentoo-dev 2004-08-29 20:23:47 UTC
glaxium defines a bunch of prototypes that the headers are forced to define

normally glaxium would define the prototypes and follow the code path opposite of what is currently forced

either way, the headers shouldnt be screwed up like this becuase who knows what other applications we broke (things that arent in portage; perhaps projects that people have made themselves)
Comment 3 Andrew Bevitt 2004-08-30 04:15:52 UTC
Reversed the patch, now glext.h is only included, GL_GLEXT_PROTOTYPES is not defined...
Comment 4 SpanKY gentoo-dev 2004-08-30 11:15:40 UTC
mmm just checked cvs and the define is still there ?

nvidia-glx/files/1.0.6111/NVIDIA_glx-1.0.6111-glheader.patch
Comment 5 Andrew Bevitt 2004-08-30 15:54:44 UTC
Vapier : yes the patch is still there, but all it does is now include GL/glext.h

Or is the point that you want glext.h not included?
Comment 6 Andrew Bevitt 2004-09-09 17:17:59 UTC
Well i didnt commit before (but we know that), this was fixed up a while back now

Closing.

glext.h should be included from gl.h