Create a new variable USE_X11_LINKS that controls whether packages that

use buildlink2 and set USE_X11 will use x11-links, or if they will simply
symlink the X11 files directly into ${BUILDLINK_DIR}.  If USE_X11_LINKS is
"YES" (the default), then x11-links is used.  If USE_X11_LINKS is "NO",
then the X11 files are symlinked directly.

There is a choice here between speed and flexibility.  Using x11-links is
faster since you don't need to spend time symlinking all of the X11 files
into ${BUILDLINK_DIR}.  However, if you update your X11 installation, then
the x11-links package will likely be out-of-date.  Conversely, directly
symlinking the X11 files will protect you from changes in your X11
installation when you update, but there is a (small) time penalty to have
to do it for every package that uses X11.

3 files changed, 10 insertions, 20 deletions

diff --git a/pkgtools/x11-links/Makefile b/pkgtools/x11-links/Makefile
index fb4d9b8274c..9090ad24712 100644
--- a/pkgtools/x11-links/Makefile
+++ b/pkgtools/x11-links/Makefile
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile,v 1.17 2002/09/20 23:53:32 jlam Exp $
+# $NetBSD: Makefile,v 1.18 2002/09/27 09:22:59 jlam Exp $
 #
 # NOTE:	If you update this package, then you'll likely need to also update
 #	the x11-links dependency in mk/x11.buildlink.mk and in
@@ -21,12 +21,11 @@ WRKSRC=		${WRKDIR}
 NO_BUILD=	# defined
 NO_MTREE=	# defined
 USE_X11=	# defined
+USE_X11_LINKS=	no
 
 PLIST_SRC=		${WRKDIR}/.PLIST_SRC
 BUILDLINK_X11_SUBDIR=	share/x11-links
 
-_FOR_X11_LINKS_ONLY=-	# needed to use x11-links/buildlink.mk
-
 .include "../../mk/bsd.prefs.mk"
    
 .if !exists(${X11BASE}/include/X11/X.h)
@@ -43,9 +42,4 @@ do-install:
 		| ${SED} "s|^\./||g" | ${SORT} -u >> ${PLIST_SRC}
 	${ECHO} "@unexec ${RM} -rf %D/${BUILDLINK_X11_SUBDIR}" >> ${PLIST_SRC}
 
-.if !empty(X11BASE:M*openwin)
-.  include "openwin.buildlink2.mk"
-.else
-.  include "xfree.buildlink2.mk"
-.endif
 .include "../../mk/bsd.pkg.mk"
diff --git a/pkgtools/x11-links/openwin.buildlink2.mk b/pkgtools/x11-links/openwin.buildlink2.mk
index 1fbc4e6b899..f8d6fb70365 100644
--- a/pkgtools/x11-links/openwin.buildlink2.mk
+++ b/pkgtools/x11-links/openwin.buildlink2.mk
@@ -1,8 +1,7 @@
-# $NetBSD: openwin.buildlink2.mk,v 1.3 2002/08/26 06:28:21 jlam Exp $
+# $NetBSD: openwin.buildlink2.mk,v 1.4 2002/09/27 09:23:00 jlam Exp $
 
-.if defined(_FOR_X11_LINKS_ONLY)
-.if !defined(X11_LINKS_BUILDLINK_MK)
-X11_LINKS_BUILDLINK_MK=	# defined
+.if !defined(OPENWIN_BUILDLINK2_MK)
+OPENWIN_BUILDLINK2_MK=	# defined
 
 BUILDLINK_PACKAGES+=		openwin
 BUILDLINK_PREFIX.openwin=	${X11BASE}
@@ -1011,5 +1010,4 @@ BUILDLINK_TARGETS+=		openwin-buildlink
 
 openwin-buildlink: _BUILDLINK_USE
 
-.endif	# X11_LINKS_BUILDLINK_MK
-.endif	# _FOR_X11_LINKS_ONLY
+.endif	# OPENWIN_BUILDLINK2_MK
diff --git a/pkgtools/x11-links/xfree.buildlink2.mk b/pkgtools/x11-links/xfree.buildlink2.mk
index 848ff379b5d..40e6dae7c6a 100644
--- a/pkgtools/x11-links/xfree.buildlink2.mk
+++ b/pkgtools/x11-links/xfree.buildlink2.mk
@@ -1,8 +1,7 @@
-# $NetBSD: xfree.buildlink2.mk,v 1.3 2002/08/26 06:28:21 jlam Exp $
+# $NetBSD: xfree.buildlink2.mk,v 1.4 2002/09/27 09:23:00 jlam Exp $
 
-.if defined(_FOR_X11_LINKS_ONLY)
-.if !defined(X11_LINKS_BUILDLINK_MK)
-X11_LINKS_BUILDLINK_MK=	# defined
+.if !defined(XFREE_BUILDLINK2_MK)
+XFREE_BUILDLINK2_MK=	# defined
 
 BUILDLINK_PACKAGES+=	XFree
 BUILDLINK_PREFIX.XFree=	${X11BASE}
@@ -545,5 +544,4 @@ _REQUIRE_BUILTIN_XPM=		YES
 
 XFree-buildlink: _BUILDLINK_USE
 
-.endif	# X11_LINKS_BUILDLINK_MK
-.endif	# _FOR_X11_LINKS_ONLY
+.endif	# XFREE_BUILDLINK2_MK