Re: CVS commit: pkgsrc/graphics/MesaLib

[David Sainty <David.Sainty%dtsp.co.nz@localhost>]

Blair Sadewitz wrote:
> David Sanity's workaround seems like a good idea to me.
I've seen that spelling more than once :)

I've committed a fix that is slightly different.  Instead of linking in 
-lpthread under NetBSD I special case NetBSD - which means the status 
quo is maintained.  I think that's probably a more elegant fix than 
dragging NetBSD down to the level of the greatest common denominator.

But I've also run NetBSD with the old patch for a long time with no 
signs of problems - we don't NEED to not link it in, NetBSD works both ways.

>   However, I'd still like to understand exactly what this bug is (if indeed 
> there is only one).
The problem is that libGL ends up referencing undefined symbols.  They 
need a definition from somewhere.  Under old NetBSD that "somewhere" is 
"libpthstub".  Under new NetBSD (+3.x) it's "libc".  Under Linux (and 
probably almost everywhere else) the only convenient place to get them 
is "libpthread".

For example, the glx-utils bombed out (in the past) with:

/tmp/pkgsrc3/graphics/glx-utils/work/.buildlink/lib/libGL.so: undefined 
reference to `pthread_getspecific'
/tmp/pkgsrc3/graphics/glx-utils/work/.buildlink/lib/libGL.so: undefined 
reference to `pthread_once'
/tmp/pkgsrc3/graphics/glx-utils/work/.buildlink/lib/libGL.so: undefined 
reference to `pthread_key_create'
/tmp/pkgsrc3/graphics/glx-utils/work/.buildlink/lib/libGL.so: undefined 
reference to `pthread_setspecific'
collect2: ld returned 1 exit status
*** Error code 1

The bzflag build detected problems (in the past) at compile time:

checking for SDL_image library... yes
configure: WARNING: Client build has been requested, but GL is not fully 
available (missing gl.h)
    ... disabling client generation

(NB: It's wrong about gl.h - configure is breaking because of link time 
errors and assumes a missing gl.h)

> OT: Getting the drm working properly has more uses than simply games:
>   

Of coooourse not.  We all appreciate your work :)

> Additionally, the drm obviates the need for opening /dev/mem 

Nice...