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

squashed ethcore-network changes which introduce error-chain #7040

Merged
merged 2 commits into from
Nov 15, 2017

Conversation

debris
Copy link
Collaborator

@debris debris commented Nov 13, 2017

squashed #6979

@debris debris added A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust. labels Nov 13, 2017
@5chdn 5chdn added this to the 1.9 milestone Nov 13, 2017
@debris debris force-pushed the squashed_network_error_chain branch from f97f5a9 to 3cf52da Compare November 13, 2017 15:49
@debris
Copy link
Collaborator Author

debris commented Nov 14, 2017

@arkpar can you look at this pr? especially last commit. Without static linking it was simply not working :/

@debris
Copy link
Collaborator Author

debris commented Nov 14, 2017

Can we merge it as it is right now? Or is there any reason why snappy shouldn't be statically linked?

@arkpar arkpar added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Nov 15, 2017
@debris debris merged commit 0230a44 into master Nov 15, 2017
@debris debris deleted the squashed_network_error_chain branch November 15, 2017 17:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants