Skip to content

doxygen: Disable unnecessary graphviz dependency by default.#2323

Merged
becker33 merged 1 commit intospack:developfrom
citibeth:efischer/161112-SimplifyDoxygen
Dec 2, 2016
Merged

doxygen: Disable unnecessary graphviz dependency by default.#2323
becker33 merged 1 commit intospack:developfrom
citibeth:efischer/161112-SimplifyDoxygen

Conversation

@citibeth
Copy link
Copy Markdown
Member

Most users just want to see HTML docs, Graphviz is not needed. And it pulls in a whole SLEW of dependencies, some of which could break. (libxcb, for example, has a Python2 build dependency, which currently breaks Python3 builds).

@becker33
Copy link
Copy Markdown
Member

@davydden You added the graphviz variant to doxygen. I think @citibeth is probably correct that this should be default off, do you have any objection to that?

Copy link
Copy Markdown
Member

@davydden davydden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that's fine with me.

@becker33 becker33 merged commit 18878a8 into spack:develop Dec 2, 2016
citibeth pushed a commit to citibeth/spack that referenced this pull request Dec 4, 2016
kserradell pushed a commit to kserradell/spack that referenced this pull request Dec 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants