Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Provide dot path to plantuml #617

Merged
merged 1 commit into from
Oct 24, 2017
Merged

Conversation

wich
Copy link
Contributor

@wich wich commented Oct 23, 2017

If HAVE_DOT = YES and DOT_PATH is configured, pass the path to the dot executable to plantuml with the -graphvizdot option.

Rationale is that plantuml does not look for dot on the path, but assumes dot is at /usr/bin/dot. The changes allow dot to be in a non-standard location which is configured in the Doxyfile.

If HAVE_DOT = YES and DOT_PATH is configured, pass the path to the dot executable to plantuml with the -graphvizdot option.
@albert-github
Copy link
Collaborator

Is it guaranteed that the dotpath has a directory separator as last character (as dot is directly appended to it)?

@wich
Copy link
Contributor Author

wich commented Oct 24, 2017

Yes, it is guaranteed that it has a directory separator as last character, this is taken care of in Config::checkAndCorrect in configimpl.l. See also dot.cpp where dot is also directly appended to the config string.

@doxygen doxygen merged commit 1308178 into doxygen:master Oct 24, 2017
@wich wich deleted the plantuml-dot-path branch October 25, 2017 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants