Skip to content
This repository was archived by the owner on Mar 13, 2023. It is now read-only.

BEEFY integration #746

Closed
wants to merge 10 commits into from
Closed

BEEFY integration #746

wants to merge 10 commits into from

Conversation

hujw77
Copy link
Contributor

@hujw77 hujw77 commented Aug 13, 2021

No description provided.

@aurexav
Copy link
Member

aurexav commented Aug 13, 2021

Need help?

@hackfisher
Copy link
Contributor

@hujw77 @xiaoch05 Prefer to use https://github.com/paritytech/grandpa-bridge-gadget (Or its fork if necessary) as dependency instead of copy to the common here.

@hujw77
Copy link
Contributor Author

hujw77 commented Aug 19, 2021

Need help?

https://github.com/paritytech/substrate/blob/287b5c5a8fe47beacfdfdf764b632b897691e9da/primitives/core/src/ecdsa.rs#L576
BEEFY use this function to verify sign, It needs the latest substrate version, Darwinia's substrate lib may need to upgrade.

@hujw77
Copy link
Contributor Author

hujw77 commented Aug 19, 2021

@hujw77 @xiaoch05 Prefer to use https://github.com/paritytech/grandpa-bridge-gadget (Or its fork if necessary) as dependency instead of copy to the common here.

OK

@aurexav
Copy link
Member

aurexav commented Aug 19, 2021

We fall behind the latest substrate codebase around 5-6 times runtime upgrade, in progress

@aurexav
Copy link
Member

aurexav commented Dec 14, 2021

@aurexav aurexav closed this Dec 14, 2021
@aurexav aurexav deleted the beefy branch January 25, 2022 02:28
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.

4 participants