Skip to content
This repository has been archived by the owner on Feb 7, 2018. It is now read-only.

Added info for generating dependancy graph on OSX #119

Merged
merged 2 commits into from
Jan 12, 2016

Conversation

ddaws
Copy link
Contributor

@ddaws ddaws commented Jan 9, 2016

It wasn't immediately obvious to me that tred and dot are from the graphviz package, and these executable are not distributed with OS X.

…ly this will save a little searching for people in the future.
@bobsummerwill
Copy link
Contributor

Yep - makes sense!

@@ -0,0 +1,10 @@

To generate the `dependancy_graph.svg` the executables `tred` and `dot` are required from the
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the file really named like that? (The name of the directory is different)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new README.md, specifically inside the dependendency_graph sub-directory, next to the python file and shell script. https://github.com/ethereum/webthree-umbrella/tree/develop/dependency_graph

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am talking about dependancy_graph.svg, I think it should be dependency_graph.svg.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yes, you are right. Sorry - didn't spot that typo.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry about that. The typo has been corrected.

@chriseth
Copy link
Contributor

Thanks for respecting pedantry!

chriseth added a commit that referenced this pull request Jan 12, 2016
Added info for generating  dependancy graph on OSX
@chriseth chriseth merged commit 8ce792f into ethereum:develop Jan 12, 2016
@bobsummerwill
Copy link
Contributor

Precision, @chriseth, precision :-)
Thanks for the merge.

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

Successfully merging this pull request may close these issues.

3 participants