ports/158723: [PATCH] lang/sbcl: don't require graphviz if not needed for build

[Quentin Stievenart]

>Number:         158723
>Category:       ports
>Synopsis:       [PATCH] lang/sbcl: don't require graphviz if not needed for build
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Fri Jul 08 08:50:01 UTC 2011
>Closed-Date:
>Last-Modified:
>Originator:     Quentin Stievenart
>Release:        FreeBSD 8.2-STABLE i386
>Organization:
>Environment:
System: FreeBSD daemon 8.2-STABLE FreeBSD 8.2-STABLE #5: Tue Mar  1 19:45:55 CET
>Description:
graphviz (which adds a lot of dependencies, especially on x11-less servers) should not be required if the user doesn't want to build sbcl's doc (ie. NOPORTDOCS is set, and the PDF and PS options aren't selected). This patch fixes that.
>How-To-Repeat:

>Fix:


Patch attached with submission follows:

diff -ruN --exclude=CVS /usr/ports/lang/sbcl/Makefile /tmp/sbcl/Makefile
--- /usr/ports/lang/sbcl/Makefile	2011-06-18 18:13:31.000000000 +0200
+++ /tmp/sbcl/Makefile	2011-07-08 09:41:57.000000000 +0200
@@ -23,8 +23,6 @@
 MAINTAINER=	stas at FreeBSD.org
 COMMENT=	A Common Lisp development system derived from the CMU CL system
 
-BUILD_DEPENDS=	dot:${PORTSDIR}/graphics/graphviz
-
 # More platforms are supported, but on Linux.
 ONLY_FOR_ARCHS=	i386 amd64
 ONLY_FOR_ARCHS_REASON=	is a native code compiler, and has not been ported to this architecture yet
@@ -90,6 +88,10 @@
 
 .include <bsd.port.pre.mk>
 
+.if !defined(NOPORTDOCS) || !defined(WITHOUT_PDF) || !defined(WITHOUT_PS)
+BUILD_DEPENDS=	dot:${PORTSDIR}/graphics/graphviz
+.endif
+
 .if !defined(WITHOUT_PDF)
 BUILD_DEPENDS+=	latex:${PORTSDIR}/print/teTeX-base \
 		texi2dvi:${PORTSDIR}/print/texinfo


>Release-Note:
>Audit-Trail:
>Unformatted: