-
Notifications
You must be signed in to change notification settings - Fork 373
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
Avoid circular dependency #180
Conversation
I am having conflicts now between dependencies as some install 6.4.1 and the fix by @xg-wang is for 6.4.0. Could we merge this to regain sanity? Elliptic is a subdependency of my project and I don't have direct control over it. Thanks a lot |
@lemoustachiste you can try to install required elliptic version as a dependency of your project and then run |
yes that's what I am doing now and it works fine. But it's a bit tedious. |
@indutny friendly reminder that PR is ready to be merged :) |
Landed in dbe8bc6 and released in 6.5.0. Thank you, and sorry for delay! |
This version uses elliptic with fixed circular dependency. Because of that circular dependency it wasn't possible to use ethereumjs-util with rollup build tool Reference: indutny/elliptic#180
Here are changes from #157 rebased to master