[1] Make the HTML manpages get put into the packages again.

[2] Check that XFREE86_VERSION=4 is set when building, rather than setting it.
[3] Move many common Makefile bits to XFree86-4-libraries/Makefile.inc
[4] Make the XFree86-4-Server port not install libXfont.so.
[5] Move the xcursor.pc file to its proper location in ${X11BASE}/libdata/pkgconfig
[6] Don't explicitly link libGL.so against libc, for the sake of 4.x's pthreads.

Submitted by:	[5,6] Ports Fury

6 files changed, 53 insertions, 19 deletions

diff --git a/x11/XFree86-4-libraries/Makefile b/x11/XFree86-4-libraries/Makefile
index 48886f05122f..088391e5a38a 100644
--- a/x11/XFree86-4-libraries/Makefile
+++ b/x11/XFree86-4-libraries/Makefile
@@ -7,7 +7,7 @@
 
 PORTNAME=	libraries
 PORTVERSION=	4.3.0
-PORTREVISION=	3
+PORTREVISION=	4
 CATEGORIES=	x11
 MASTER_SITES=	${MASTER_SITE_XFREE:S/$/:x/} \
 		${MASTER_SITE_LOCAL:S/$/:local/}
@@ -25,23 +25,14 @@ EXTRACT_ONLY=	X430src-1.tgz \
 MAINTAINER=	anholt@freebsd.org
 COMMENT=	XFree86-4 libraries and headers
 
-BUILD_DEPENDS=	${X11BASE}/lib/X11/config/date.def:${PORTSDIR}/devel/imake-4
-LIB_DEPENDS=	freetype.9:${PORTSDIR}/print/freetype2
-LIB_DEPENDS=	fontconfig.1:${PORTSDIR}/x11-fonts/fontconfig
-RUN_DEPENDS=	mkhtmlindex:${PORTSDIR}/devel/imake-4
+LIB_DEPENDS=	freetype.9:${PORTSDIR}/print/freetype2 \
+		fontconfig.1:${PORTSDIR}/x11-fonts/fontconfig
 
-XFREE86_VERSION=	4
+USE_X_PREFIX=		no
 PREFIX?=		${X11BASE}
-MANCOMPRESSED=		yes
-XFREE86_HTML_MAN=	yes
 MTREE_FILE=		/etc/mtree/BSD.x11-4.dist
 INSTALLS_SHLIB=		yes
-DIST_SUBDIR=		xc
-WRKSRC=			${WRKDIR}/xc
-SCRIPTS_ENV=		DebuggableLibraries=${DebuggableLibraries} \
-			CC="${CC}" \
-			CXX="${CXX}" \
-			CFLAGS="${CFLAGS}"
+SCRIPTS_ENV=		DebuggableLibraries=${DebuggableLibraries}
 MAN3=			Xaw.3 Xcursor.3 Xrandr.3
 XBUILD_DIRS=		lib nls programs/Xserver/include \
 			programs/Xserver/hw/xfree86/parser
@@ -60,6 +51,7 @@ DebuggableLibraries?=	NO
 .error You have `USE_XLIB' variable defined either in environment or in make(1) arguments. Please undefine and try again.
 .endif
 
+.include "Makefile.inc"
 .include <bsd.port.pre.mk>
 
 # BuildXF86DRI is false for FreeBSD < 4.1
@@ -88,6 +80,8 @@ post-install:
 		${PREFIX}/include/X11/PM
 	${INSTALL_DATA} ${WRKSRC}/programs/proxymngr/PMproto.h \
 		${PREFIX}/include/X11/PM
+	${MKDIR} ${PREFIX}/libdata/pkgconfig
+	${INSTALL_DATA} ${WRKSRC}/lib/Xcursor/xcursor.pc \
+		${PREFIX}/libdata/pkgconfig
 
-.include "Makefile.inc"
 .include <bsd.port.post.mk>
diff --git a/x11/XFree86-4-libraries/Makefile.inc b/x11/XFree86-4-libraries/Makefile.inc
index 5eb9f2e41002..9fefaec4006c 100644
--- a/x11/XFree86-4-libraries/Makefile.inc
+++ b/x11/XFree86-4-libraries/Makefile.inc
@@ -1,4 +1,7 @@
-# include for XFree86-4 ports which share XFree86-4-libraries's patches.
+# Include before bsd.port.pre.mk for XFree86-4 ports which share 
+# XFree86-4-libraries's patches.
+# The purpose of this file is to reduce the duplicated code among the XFree86-4
+# ports which each do only part of a standard XFree86 build.
 #
 # XBUILD_DIRS is the group of directories under ${WRKSRC} that will 
 #   be built in.
@@ -9,11 +12,26 @@
 # XINSTALL_MAN_DIRS is the group of directories which will have 
 #   their install.man targets run.
 
+DIST_SUBDIR=	xc
+WRKSRC?=	${WRKDIR}/xc
+USE_IMAKE=	YES
+XFREE86_HTML_MAN=	yes
+PATCHDIR=	${.CURDIR}/../../x11/XFree86-4-libraries/files
+SCRIPTS_ENV+=	CC="${CC}" \
+		CXX="${CXX}" \
+		CFLAGS="${CFLAGS}"
 MAKE_ENV+=	FBSDCC="${CC}" FBSDCXX="${CXX}"
+
 _XBUILD_DIRS=	include ${XBUILD_DIRS}
 XINSTALL_DIRS?=	${XBUILD_DIRS}
 _XINCLUDE_DIRS=	${_XBUILD_DIRS} ${XINCLUDE_DIRS}
-XFREE86_HTML_MAN?= no
+
+pre-everything::
+	@if [ "x${XFREE86_VERSION}" != "x4" ]; then \
+		${ECHO_MSG} "If you use the XFree86-4 ports you must have XFREE86_VERSION=4"; \
+		${ECHO_MSG} "set in your /etc/make.conf."; \
+		${FALSE}; \
+	fi;
 
 .if !target(do-configure)
 do-configure:
diff --git a/x11/XFree86-4-libraries/files/patch-Xcursor-Imakefile b/x11/XFree86-4-libraries/files/patch-Xcursor-Imakefile
new file mode 100644
index 000000000000..c9c5ace3f899
--- /dev/null
+++ b/x11/XFree86-4-libraries/files/patch-Xcursor-Imakefile
@@ -0,0 +1,11 @@
+--- lib/Xcursor/Imakefile.orig	Tue May 13 14:34:23 2003
++++ lib/Xcursor/Imakefile	Tue May 13 14:34:34 2003
+@@ -69,8 +69,6 @@
+ 	RemoveFile($@)
+ 	sh config-subst $(SUBSTVARS) < xcursor.pc.in > $@
+ 
+-InstallNonExecFile(xcursor.pc,$(USRLIBDIR)/pkgconfig)
+-
+ clean::
+ 	RemoveFile(xcursor.pc)
+ 
diff --git a/x11/XFree86-4-libraries/files/patch-bsdLib.tmpl b/x11/XFree86-4-libraries/files/patch-bsdLib.tmpl
new file mode 100644
index 000000000000..9da65d75e70f
--- /dev/null
+++ b/x11/XFree86-4-libraries/files/patch-bsdLib.tmpl
@@ -0,0 +1,11 @@
+--- config/cf/bsdLib.tmpl.orig	Tue May 13 12:37:22 2003
++++ config/cf/bsdLib.tmpl	Tue May 13 12:37:23 2003
+@@ -65,7 +65,7 @@
+ #define SharedXieReqs	$(LDPRELIBS) $(XLIBONLY)
+ #define SharedXvReqs	$(LDPRELIB) $(XLIB)
+ #define SharedSMReqs	$(LDPRELIBS) $(ICELIB)
+-#define SharedGLReqs	$(LDPRELIBS) $(XLIB) -lc
++#define SharedGLReqs	$(LDPRELIBS) $(XLIB)
+ #define SharedGLUReqs	$(LDPRELIB) $(XLIB) $(GLXLIB)
+ #define SharedXcursorReqs   $(LDPRELIB) $(XRENDERLIB) $(XLIB)
+ 
diff --git a/x11/XFree86-4-libraries/files/patch-z34 b/x11/XFree86-4-libraries/files/patch-z34
index 71bc523266e1..747d662b3c7c 100644
--- a/x11/XFree86-4-libraries/files/patch-z34
+++ b/x11/XFree86-4-libraries/files/patch-z34
@@ -5,7 +5,7 @@
  #endif
  
 -#if BuildLibraries
-+#if BuildLibraries && !FreeBSDBuildXbin
++#if BuildLibraries && FreeBSDBuildXlib
  #define LibInstall YES
  #else
  #define LibInstall NO
diff --git a/x11/XFree86-4-libraries/pkg-plist b/x11/XFree86-4-libraries/pkg-plist
index 5d6d16b4cbd0..4bfef5c40ef8 100644
--- a/x11/XFree86-4-libraries/pkg-plist
+++ b/x11/XFree86-4-libraries/pkg-plist
@@ -619,4 +619,4 @@ lib/libpsres.so.1
 lib/libxf86config.a
 lib/libxkbfile.a
 lib/libxkbui.a
-lib/pkgconfig/xcursor.pc
+libdata/pkgconfig/xcursor.pc