Skip to content
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

Patch release Polkadot v1.2.1 Rococo & Westend runtimes that include crates name fix #2085

Closed

Conversation

acatangiu
Copy link
Contributor

This is a backport of #1948 over Polkadot v1.2.0 release.

102000 Rococo & Westend runtimes are breaking ecosystem apps because of inner objects renaming (staging-*). This PR applies the fix for that and bumps specs to 102001 to be released in patch release v1.2.1.

bkchr and others added 3 commits October 30, 2023 16:04
)

The `xcm` crate was renamed to `staging-xcm` to be able to publish it to
crates.io as someone as squatted `xcm`. The problem with this rename is
that the `TypeInfo` includes the crate name which ultimately lands in
the metadata. The metadata is consumed by downstream users like
`polkadot-js` or people building on top of `polkadot-js`. These people
are using the entire `path` to find the type in the type registry. Thus,
their code would break as the type path would now be [`staging_xcm`,
`VersionedXcm`] instead of [`xcm`, `VersionedXcm`]. This pull request
fixes this by renaming the path segment `staging_xcm` to `xcm`.

This requires: paritytech/scale-info#197

---------

Co-authored-by: Francisco Aguirre <franciscoaguirreperez@gmail.com>
@acatangiu acatangiu added the T4-runtime_API This PR/Issue is related to runtime APIs. label Oct 30, 2023
@acatangiu acatangiu self-assigned this Oct 30, 2023
@acatangiu acatangiu requested review from a team October 30, 2023 14:15
@acatangiu acatangiu requested a review from a team October 30, 2023 14:15
@acatangiu acatangiu requested a review from a team as a code owner October 30, 2023 14:15
@acatangiu acatangiu requested review from koute and a team as code owners October 31, 2023 10:39
@acatangiu
Copy link
Contributor Author

Holding off on this - we might release 1.3.0 following days and we can use that directly

@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable 1/3
Logs: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/4147459

@coderobe
Copy link
Contributor

coderobe commented Nov 7, 2023

1.3.0 is out which should have the fix

@coderobe coderobe closed this Nov 7, 2023
@acatangiu acatangiu deleted the polkadot-v1.2.1 branch April 18, 2024 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T4-runtime_API This PR/Issue is related to runtime APIs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants