Re: pkg/44792: graphics/magicpoint build problem with X11_TYPE == "modular"?

[Joerg Sonnenberger <joerg%britannica.bec.de@localhost>]

The following reply was made to PR pkg/44792; it has been noted by GNATS.

From: Joerg Sonnenberger <joerg%britannica.bec.de@localhost>
To: Thomas Klausner <wiz%NetBSD.org@localhost>
Cc: Takahiro Kambe <taca%NetBSD.org@localhost>, gnats-bugs%NetBSD.org@localhost
Subject: Re: pkg/44792: graphics/magicpoint build problem with X11_TYPE ==
 "modular"?
Date: Wed, 30 Mar 2011 13:22:52 +0200

 On Wed, Mar 30, 2011 at 10:06:59AM +0200, Thomas Klausner wrote:
 > On Wed, Mar 30, 2011 at 05:01:59PM +0900, Takahiro Kambe wrote:
 > > In message <20110330073502.23E0163B898%www.NetBSD.org@localhost>
 > >    on Wed, 30 Mar 2011 07:35:02 +0000 (UTC),
 > >    Thomas Klausner <wiz%NetBSD.org@localhost> wrote:
 > > >  On Wed, Mar 30, 2011 at 06:25:01AM +0000, taca%NetBSD.org@localhost 
 > > > wrote:
 > > >  > In file included from imlib_loader.c:8:
 > > >  > 
 > > > /pkg_comp/obj/pkgsrc/graphics/magicpoint/default/.buildlink/include/X11/extensio
 > > >  > ns/shape.h:34:39: error: X11/extensions/shapeconst.h: No such file or 
 > > > directory
 > > >  > *** [imlib_loader.o] Error code 1
 > > >  
 > > >  What provides X11/extensions/shape.h?
 > > x11/xextproto
 > 
 > No, actually, libXext provides that.
 > And its bl3.mk says:
 > # XXX don't include xextproto/bl3 here; packages checking for xext.pc must
 > # include it theirselfes
 > 
 > I don't think that this is appropriate, since libXext's shape.h can
 > pull in xextproto's shapeconst.h.
 > 
 > Drochner wanted to add this inclusion, but joerg asked him to back it
 > out.
 > 
 > Joerg, can you please explain again why it shouldn't be included here?
 > Or can we include it now?
 
 See last commit message?
 
 Joerg