Re: cairo 1.12.4 and 1.12.6 fail to build on OSX 10.6.8

On Wed, 31 Oct 2012 14:11:05 +0100, Thomas Klausner <[email protected]> wrote:
> On Sun, Oct 28, 2012 at 10:15:50AM +0000, Chris Wilson wrote:
> > Thank you for your report. I've included the necessary checks in
> > configure to detect which version of the headers to use:
> > 
> > commit f18199fcfb3b052c198041fc05156ae3d9e6aee7
> > Author: Chris Wilson <[email protected]>
> > Date:   Sun Oct 28 10:04:12 2012 +0000
> > 
> >     xlib/shm: Check for XShm headers
> >     
> >     Not all version of libXext ship the same set of headers, so play safe
> >     and check during configure that we have the headers we depend upon in
> >     the code.
> >     
> >     Reported-by: Sebastian Haas <[email protected]>
> >     Signed-off-by: Chris Wilson <[email protected]>
> > 
> > commit 0e2458697848cf8c89c9d57fa9b64f0ea7bd0877
> > Author: Chris Wilson <[email protected]>
> > Date:   Sun Oct 28 10:08:39 2012 +0000
> > 
> >     xlib/shm: Use shmstr.h instead of shmproto.h if available
> >     
> >     Before it was known as shmproto.h, the wire protocol definition was to
> >     be found in shmstr.h, so if we don't have the current version of libXext
> >     try to use the older includes.
> >     
> >     Reported-by: Sebastian Haas <[email protected]>
> >     Signed-off-by: Chris Wilson <[email protected]>
> 
> I've added these patches to pkgsrc to get further in the build
> breakage on NetBSD-5[1], but this breaks startup of firefox and others on
> NetBSD-6 with:
> assertion "!"reached"" failed: file "cairo-xlib-surface-shm.c", line 58,
> function "_cairo_xlib_surface_put_shm"

Thanks, that was a mistake on part thinking that it wouldn't be called
if we never succeeded in getting a shm surface.
 
> The programs work fine with cairo-1.12.6 + the "Xorg"-recognition fix
> but without those patches.

Right, this is worrying as we know believe that we can't enable SHM on
your system. Can you please attach the config.log?

>  Thomas
> 
> [1] The build on NetBSD-5 later fails with
>     CC     cairo-gl-device.lo
>   cairo-gl-device.c: In function '_cairo_gl_context_init':
>   cairo-gl-device.c:285: error: 'GL_MAX_RENDERBUFFER_SIZE' undeclared (first use in this function)

Looks like we need to be testing for a recent enough version of glext.h
before allowing the gl backend to be built. You will need to be careful
to use --enable-gl=auto if you want to use the same pkg script and not
have configure error out if it cannot build GL when passed --enable-gl.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
-- 
cairo mailing list
[email protected]
http://lists.cairographics.org/mailman/listinfo/cairo