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

Add warning when GraphViz is not installed #4

Closed
clue opened this issue May 21, 2013 · 2 comments · Fixed by #43
Closed

Add warning when GraphViz is not installed #4

clue opened this issue May 21, 2013 · 2 comments · Fixed by #43

Comments

@clue
Copy link
Owner

clue commented May 21, 2013

Currently, generating a graph image just fails if GraphViz (dot executable) is not installed. Instead, we should check for the existence and fail gracefully with a concise, readable error message. On Debian-based systems, we could perhaps also offer to install it via sudo apt-get install graphviz.

@fzaninotto
Copy link

Agreed, the current error message:

Unable to invoke "dot" to create image file  (code 127)

Doesn't speak to those who don't know that dot is GraphViz. You should also update the installation manual to explain that downloading the phar is not enough.

@clue
Copy link
Owner Author

clue commented Sep 13, 2013

You should also update the installation manual

Indeed, thanks for commenting! As a first step, I've just updated the install instructions (d322855).

Anyway, I'll leave this open until we fail gracefully with a concise, readable error message.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants