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

markdown plantuml use of graphviz fails in plantuml, works in code file (Origin: bugzilla #792449) #6312

Closed
doxygen opened this issue Jul 2, 2018 · 2 comments
Labels

Comments

@doxygen
Copy link
Owner

doxygen commented Jul 2, 2018

status NEW severity major in component doxywizard for ---
Reported in version 1.8.14 on platform Other
Assigned to: Dimitri van Heesch

Original attachment names and IDs:

On 2018-01-12 06:41:25 +0000, rasmusms@gmail.com wrote:

Created attachment 366700
doxygen file for c plantuml to work

Hi
I have a project with c and markdown files. I would like both file types to
support plantUML. I can get one of them to work at a time, but need to
change the path to graphviz. For c files it have to be C:/Program Files
(x86)/Graphviz2.38/bin and for markdown it have to be C:/Program Files
(x86)/Graphviz2.38/lib/release/lib. So it seems they do not use same dot
entry point. One is exe and other is lib. If not correct path for c file I
do not get images of dependencies and if for markdown "Dot executable:
C:\program files (x86)\graphviz2.38\bin\dot file does not exist Cannot find
graphviz. You should try @startuml testdot @enduml"

On 2018-01-31 16:32:33 +0000, Mira Kes wrote:

I can confirm this problem.

I tried some debugging and the problem is that the plantuml.jar wants the
full name of the dot executable i.e. (on Windows) not just "dot" but
"dot.exe" ...

java -jar plantuml.jar -graphvizdot "C:\Program Files
(x86)\Graphviz2.38\bin\dot" -testdot
The environment variable GRAPHVIZ_DOT has been set to C:\Program Files
(x86)\Graphviz2.38\bin\dot
Dot executable is C:\Program Files (x86)\Graphviz2.38\bin\dot
Error: File does not exist
Error: only sequence diagrams will be generated

java -jar plantuml.jar -graphvizdot "C:\Program Files
(x86)\Graphviz2.38\bin\dot.exe" -testdot
The environment variable GRAPHVIZ_DOT has been set to C:\Program Files
(x86)\Graphviz2.38\bin\dot.exe
Dot executable is C:\Program Files (x86)\Graphviz2.38\bin\dot.exe
Dot version: dot - graphviz version 2.38.0 (20140413.2041)

The problem is that this patch:
https://github.com/doxygen/doxygen/commit/SHA:
c24cb9a74ef0af854455047f29f9925d79ac0195

doesn't handle the dot executable name on Windows the way the plantuml
requires.

On 2018-06-12 13:38:29 +0000, albert wrote:

I didn't completely get the statements the original bug, but the comment
gave the right pointer.

I hope I also corrected the title properly.

I've just pushed a proposed patch to github (pull request 764,
https://github.com/doxygen/doxygen/pull/764).
@doxygen doxygen added the doxywizard bug is specific for the wizard label Jul 19, 2018
@albert-github albert-github added bug fixed but not released Bug is fixed in github, but still needs to make its way to an official release and removed doxywizard bug is specific for the wizard labels Jul 24, 2018
@albert-github
Copy link
Collaborator

Code has been integrated in master on github.

@doxygen
Copy link
Owner Author

doxygen commented Dec 28, 2018

This issue was previously marked 'fixed but not released',
which means it should be fixed in doxygen version 1.8.15.
Please verify if this is indeed the case. Reopen the
issue if you think it is not fixed and please include any additional information
that you think can be relevant (preferably in the form of a self-contained example).

@doxygen doxygen removed the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label Dec 28, 2018
@doxygen doxygen closed this as completed Dec 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants