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

Use coherent prost crate versions #9676

Merged
merged 1 commit into from
Sep 2, 2021
Merged

Use coherent prost crate versions #9676

merged 1 commit into from
Sep 2, 2021

Conversation

adoerr
Copy link
Contributor

@adoerr adoerr commented Sep 2, 2021

We were mixing 0.7.0 and 0.8.0 versions of the prost and prost-build crates.

Version 0.7.0 is causing some clippy CI trouble downstream.

No code changes

@adoerr adoerr added A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit B0-silent Changes should not be mentioned in any release notes labels Sep 2, 2021
@adoerr adoerr changed the title Use coherent prost crate versions Use coherent prost crate versions Sep 2, 2021
Copy link
Contributor

@gilescope gilescope left a comment

Choose a reason for hiding this comment

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

LGTM - Thank you!

@adoerr
Copy link
Contributor Author

adoerr commented Sep 2, 2021

syncs_header_only_forks() test fails for gitlab-test-linux-stable but works perfectly fine locally every time. Any ideas, what this could be 🤔

Also, how can I re-run a Gitlab task?

@waynenilsen
Copy link

this has become slightly more important now that 0.7 does not build with nightly

Copy link
Contributor

@KiChjang KiChjang left a comment

Choose a reason for hiding this comment

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

Double-checked with my local clone of substrate, looks like we got all places where we import prost or prost-build to be at 0.8, let's ship it!

@KiChjang KiChjang added A8-mergeoncegreen and removed A0-please_review Pull request needs code review. labels Sep 2, 2021
@gilescope gilescope merged commit dd29788 into master Sep 2, 2021
@gilescope gilescope deleted the ad-prost branch September 2, 2021 18:50
@dvdplm
Copy link
Contributor

dvdplm commented Sep 3, 2021

Came here to ask for this and then it happened all by itself. ❤️

chevdor pushed a commit that referenced this pull request Sep 6, 2021
chevdor pushed a commit that referenced this pull request Sep 6, 2021
chevdor pushed a commit that referenced this pull request Sep 6, 2021
nuke-web3 pushed a commit that referenced this pull request Sep 6, 2021
chevdor pushed a commit that referenced this pull request Sep 7, 2021
bkchr added a commit that referenced this pull request Sep 7, 2021
* RUSTSEC-2021-0076 bump libsecp256k1 (#9391)

* RUSTSEC-2021-0076 bump libsecp256k1

libsecp256k1 allows overflowing signatures
https://rustsec.org/advisories/RUSTSEC-2021-0076

Changes were made to conform to libsecp256k1 version differences.

Closes #9356

* parse_standard_slice() -> parse_overflowing_slice()

* Added v2 host function for ecdsa_verify

* Add feature tag over helpers

* Added ecdsa_verify v2 to test runner

* PR feedback

- Spaces -> tabs
- renamed two helper functions

* Fixed imports after rebasing

* Bump rest of libsecp256k1 (and libp2p)

libp2p also uses libsecp256k1 so it is required to be bumped too, along
with all the version difference changes.

* Add version2 for ecdsa pubkey recovery

* libp2p rebase master fixes

* Fix test panic when non Behaviour event is returned

* Update bin/node/browser-testing/Cargo.toml

* Update primitives/core/src/ecdsa.rs

* Update primitives/core/src/ecdsa.rs

* Update Cargo.lock

Co-authored-by: Bastian Köcher <bkchr@users.noreply.github.com>

* Use coherent prost crate  versions (#9676)

* Bump node-browser-testing deps on prost

Co-authored-by: Trevor Arjeski <72849114+trevor-crypto@users.noreply.github.com>
Co-authored-by: Bastian Köcher <bkchr@users.noreply.github.com>
Co-authored-by: Andreas Doerr <adoerr@users.noreply.github.com>
cmichi pushed a commit that referenced this pull request Sep 9, 2021
* RUSTSEC-2021-0076 bump libsecp256k1 (#9391)

* RUSTSEC-2021-0076 bump libsecp256k1

libsecp256k1 allows overflowing signatures
https://rustsec.org/advisories/RUSTSEC-2021-0076

Changes were made to conform to libsecp256k1 version differences.

Closes #9356

* parse_standard_slice() -> parse_overflowing_slice()

* Added v2 host function for ecdsa_verify

* Add feature tag over helpers

* Added ecdsa_verify v2 to test runner

* PR feedback

- Spaces -> tabs
- renamed two helper functions

* Fixed imports after rebasing

* Bump rest of libsecp256k1 (and libp2p)

libp2p also uses libsecp256k1 so it is required to be bumped too, along
with all the version difference changes.

* Add version2 for ecdsa pubkey recovery

* libp2p rebase master fixes

* Fix test panic when non Behaviour event is returned

* Update bin/node/browser-testing/Cargo.toml

* Update primitives/core/src/ecdsa.rs

* Update primitives/core/src/ecdsa.rs

* Update Cargo.lock

Co-authored-by: Bastian Köcher <bkchr@users.noreply.github.com>

* Use coherent prost crate  versions (#9676)

* Bump node-browser-testing deps on prost

Co-authored-by: Trevor Arjeski <72849114+trevor-crypto@users.noreply.github.com>
Co-authored-by: Bastian Köcher <bkchr@users.noreply.github.com>
Co-authored-by: Andreas Doerr <adoerr@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants