Skip to content
This repository was archived by the owner on Nov 6, 2020. It is now read-only.

update install-deps.sh #316

Merged
merged 3 commits into from
Feb 3, 2016
Merged

update install-deps.sh #316

merged 3 commits into from
Feb 3, 2016

Conversation

debris
Copy link
Collaborator

@debris debris commented Feb 3, 2016

No description provided.

@debris debris added the A0-pleasereview 🤓 Pull request needs code review. label Feb 3, 2016
then
exe brew reinstall parity
else
exe brew install parity
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

explicit installing rocksdb and multirust is no needed. parity tap will take care of it

@gavofyork
Copy link
Contributor

should be different script.

@gavofyork gavofyork added A4-gotissues 💥 Pull request is reviewed and has significant issues which must be addressed. and removed A0-pleasereview 🤓 Pull request needs code review. labels Feb 3, 2016
@debris debris added A0-pleasereview 🤓 Pull request needs code review. and removed A4-gotissues 💥 Pull request is reviewed and has significant issues which must be addressed. labels Feb 3, 2016
@debris
Copy link
Collaborator Author

debris commented Feb 3, 2016

now, it's in separate file

gavofyork pushed a commit that referenced this pull request Feb 3, 2016
@gavofyork gavofyork merged commit fe97809 into master Feb 3, 2016
@gavofyork gavofyork deleted the install-deps branch February 3, 2016 14:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-pleasereview 🤓 Pull request needs code review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants