Bug 229369
| Summary: | lang/swift: Don't install sphinx unless building docs | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Adam Weinberger <adamw> | ||||
| Component: | Individual Port(s) | Assignee: | Steve Wills <swills> | ||||
| Status: | Closed FIXED | ||||||
| Severity: | Affects Only Me | Flags: | bugzilla:
maintainer-feedback?
(swills) |
||||
| Priority: | --- | ||||||
| Version: | Latest | ||||||
| Hardware: | Any | ||||||
| OS: | Any | ||||||
| Attachments: |
|
||||||
(In reply to Adam Weinberger from comment #0) Approved, thanks! A commit references this bug: Author: adamw Date: Wed Jun 27 22:18:32 UTC 2018 New revision: 473461 URL: https://svnweb.freebsd.org/changeset/ports/473461 Log: Put sphinx-build behind DOCS options, rather than bringing it in unconditionally PR: 229369 Approved by: maintainer (swills) Changes: head/lang/swift/Makefile Done. Thanks Steve! |
Created attachment 194680 [details] Conditionalize sphinx-build Hey, Steve. Right now, lang/swift depends on sphinx-build unconditionally, which brings in about 20 extra dependencies. However, it's not needed at all when DOCS is off. The attached patch makes that dependency, and the attempt to build docs, dependent on the DOCS option.