Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 8 Jul 2011 08:41:41 GMT
From:      Quentin Stievenart <acieroid@awesom.eu>
To:        freebsd-gnats-submit@FreeBSD.org
Subject:   ports/158723: [PATCH] lang/sbcl: don't require graphviz if not needed for build
Message-ID:  <201107080841.p688ffAZ052385@red.freebsd.org>
Resent-Message-ID: <201107080850.p688o18p066629@freefall.freebsd.org>

next in thread | raw e-mail | index | archive | help

>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@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:



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201107080841.p688ffAZ052385>