Jens Owen wrote: > > Brian Paul wrote: > > > > Jens Owen wrote: > > > > > > Brian Paul wrote: > > > > > > > > Jens Owen wrote: > > > > > > > > > Wasn't this stuff recently submitted to the DRI trunk? Maybe we can fix > > > > > this incompatability if this hasn't propogated out to any major distros, > > > > > yet. > > > > > > > > Yes, we should really fix it ASAP. > > > > > > Was this part of Mesa 4? So this wouldn't have gone out in XFree86 4.2 > > > or any other source releases where we care about binary compatability, > > > right? > > > > It's not a Mesa issue, it's an XFree86 libGL issue. > > > > Just remove the field from the struct and replace any occuraces of the > > field with 'currentDrawable. instead. That'll work fine. > > I've patched every occurance with the attached patch, but 3 out of 5 > occurances are from the /extras/Mesa tree. > > Please propegate this patch back to Mesa, or if you want a different > fix, propogate that patch to me so we can sync up the /extra/Mesa tree.
I should have been more clear when I wrote "it's not a Mesa issue, it's an XFree86 libGL issue". You don't have to change any Mesa code. Mesa's fakeglx.c, glxapi.c and glxapi.h are only used in XFree86 when the GlxBuiltInXMesa option is defined, and when it is, the references to currentReadable are skipped because of the GLX_BUILT_IN_XMESA cpp token being defined. Without doubt, this is somewhat confusing code. Stand-alone Mesa and XFree86's libGL have a lot of overlap and it's sometimes hard to see how the pieces interact. I'm about to check in some changes to both the DRI and Mesa trees to finish this up. Just doing some build testing now. -Brian _______________________________________________ Dri-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/dri-devel
