Re: pkg/50243: graphics/libepoxy and option EGL from MesaLib

[Tobias Nygren <tnn%NetBSD.org@localhost>]

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

From: Tobias Nygren <tnn%NetBSD.org@localhost>
To: joern.clausen%uni-bielefeld.de@localhost
Cc: gnats-bugs%NetBSD.org@localhost
Subject: Re: pkg/50243: graphics/libepoxy and option EGL from MesaLib
Date: Mon, 14 Sep 2015 15:10:35 +0200

 On Mon, 14 Sep 2015 11:25:00 +0000 (UTC)
 joern.clausen%uni-bielefeld.de@localhost wrote:
 
 >  PLIST_VARS+=           egl
 > -.if ${X11_TYPE} == "modular" || exists(${X11BASE}/include/EGL/egl.h)
 > +.if  && exists(${X11BASE}/include/EGL/egl.h)
 >  PLIST.egl=             yes
 >  CONFIGURE_ENV+=                PKGSRC_BUILD_EGL=yes
 >  .else
 > 
 > Maybe the check for modular X11 is now unnecessary?
 
 I'm afraid that is not going to work. In the modular case we cannot
 safely rely on exists() because the dependency package might not be
 installed yet in PREFIX. Maybe something like this will work:
 
 .include "../../graphics/MesaLib/buildlink3.mk"
 .if ${X11_TYPE} == "modular" && !empty(PKG_BUILD_OPTIONS.MesaLib:Mdri) \
 	|| exists(${X11BASE}/include/EGL/egl.h)
 
 These packages also use this idiom:
 multimedia/libva/Makefile
 x11/qt5-qtbase/Makefile