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
Fixes some build issues when importing flow-go
as a dependency
#1333
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Codecov Report
@@ Coverage Diff @@
## master #1333 +/- ##
==========================================
+ Coverage 56.25% 56.29% +0.03%
==========================================
Files 500 500
Lines 31140 31140
==========================================
+ Hits 17518 17529 +11
+ Misses 11253 11242 -11
Partials 2369 2369
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Of course. I wonder if we can test the crypto-version pin desync in CI. @tarakby, would you have any idea?
bors merge |
1333: Fixes some build issues when importing `flow-go` as a dependency r=jordanschalm a=jordanschalm This fixes 2 build issues when using `flow-go` as a dependency. 1. The dkg controller factory mock did not have a relic build tag set, but made use of a relic-tagged interface. Solution: Move the interface to a non-relic-tagged file. ``` ../flow-go/module/mock/dkg_controller_factory.go:18:108: undefined: module.DKGController ../flow-go/module/mock/dkg_controller_factory.go:21:9: undefined: module.DKGController ../flow-go/module/mock/dkg_controller_factory.go:22:67: undefined: module.DKGController ../flow-go/module/mock/dkg_controller_factory.go:26:21: undefined: module.DKGController ``` 2. The networking layer imported a function which did not exist in the `crypto` version pinned by `flow-go`. For local development this worked, because we replace with the local `crypto` module, but failed when used as a dependency. ``` ../flow-go/network/p2p/keyutils/keyTranslator.go:169:14: undefined: "github.com/onflow/flow-go/crypto".DecodePublicKeyCompressed ``` Co-authored-by: Jordan Schalm <jordan@dapperlabs.com>
Canceled. |
Hmmm, I just tried to run DPS with this branch as a dependency and got
|
This fixes 2 build issues when using
flow-go
as a dependency.crypto
version pinned byflow-go
. For local development this worked, because we replace with the localcrypto
module, but failed when used as a dependency.