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.

Sign up for GitHub

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

Jump to bottom

Ship graphviz? #16

Open
NHDaly opened this issue Nov 28, 2019 · 4 comments · May be fixed by #71
Open

Ship graphviz? #16

NHDaly opened this issue Nov 28, 2019 · 4 comments · May be fixed by #71
Labels
binary_dependency

Comments

@NHDaly
Copy link
Member

NHDaly commented Nov 28, 2019

We should consider shipping graphviz instead of requiring it to be installed. Maybe we can add it to Yggdrasil?
@NHDaly
Copy link
Member Author

NHDaly commented Dec 30, 2019

I've started work on this, here:
JuliaPackaging/Yggdrasil#351

@NHDaly NHDaly mentioned this issue May 4, 2020
Merged
@LilithHafner
Copy link

@Keno, are GraphVis.jl or GraphVis_jll appropriate here?

@fingolfin
Copy link

There is Graphviz_jll these days:

julia> Graphviz_jll.dot() do exe
         run(`$exe -V`)
       end
dot - graphviz version 2.50.0 (20211204.2007)
Process(`/Users/mhorn/.julia/artifacts/dbc1b7f42ddb07be07d55ad976317ad027618c63/bin/dot -V`, ProcessExited(0))

@fingolfin fingolfin linked a pull request Oct 10, 2022 that will close this issue
Open
@fingolfin
Copy link

See PR #71

@NHDaly NHDaly mentioned this issue Oct 18, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
binary_dependency
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use Graphviz_jll fingolfin/PProf.jl
3 participants
@fingolfin @NHDaly @LilithHafner