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 graphviz as dependency so pip could install it automatically #37

Open
Petrox opened this issue Oct 17, 2023 · 5 comments
Open

Add graphviz as dependency so pip could install it automatically #37

Petrox opened this issue Oct 17, 2023 · 5 comments

Comments

@Petrox
Copy link

Petrox commented Oct 17, 2023

This would solve the docker container customization with every new release overhead for any users here.

automatic apt-getting in the container env is decided to be not implemented
inventree/InvenTree#5718

@SchrodingersGat
Copy link
Member

Can this be installed via pip?

@Petrox
Copy link
Author

Petrox commented Oct 17, 2023

pip3 install graphviz
Collecting graphviz
  Using cached https://files.pythonhosted.org/packages/9d/fb/886e8ec7862989afc0c35d15813b6c665fe134cc6027cdde2fa300abe9a2/graphviz-0.19.1-py3-none-any.whl
Installing collected packages: graphviz
Successfully installed graphviz-0.19.1

@SchrodingersGat
Copy link
Member

Right, but the python graphviz package is already installed as part of wireviz

Is this not getting installed automatically?

@Petrox
Copy link
Author

Petrox commented Oct 17, 2023

TBH I didn't check if it installs it or not, I've read the readme some time ago that it depends on graphviz and I should install it.

WireWiz requires GraphViz to be installed in order to work. See the GraphViz download page for OS-specific instructions.

Maybe it is not necessary at all if graphviz (python package) is installed?

When I was using the stable container I got the error, that it could not find the graphviz binary, so it either not included in the pip package or not in PATH inside the container.

@SchrodingersGat
Copy link
Member

To be clear, the graphviz python package gets installed automatically, as part of this plugin, but you need to have the graphviz binary also installed on your system - I don't think PIP takes care of this part

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

No branches or pull requests

2 participants