Skip to content

chore: bump rust to latest stable #167

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gjermundgaraba
Copy link

No description provided.

@eloylp
Copy link

eloylp commented Apr 24, 2025

Hi @webmaster128 - apologies for the direct ping.

At Eiger, we’re currently building a Solana <-> Axelar bridge. Since the Axelar network uses CosmWasm smart contracts, we need to include certain Solana dependencies that require the latest stable Rust version.

This PR is currently a blocker for us, and we’d greatly appreciate it if it could be reviewed and merged soon. A new release including this change would unblock our integration work.

Thanks a lot in advance, and happy to provide any additional info if needed!

@chipshort
Copy link
Contributor

@eloylp This cannot currently be merged because it is blocked by CosmWasm/cosmwasm#2292. We are currently waiting for a Wasmer release that has full reference-types support because latest Rust enforces having this.

@chipshort
Copy link
Contributor

chipshort commented Apr 24, 2025

Alternatively, it might be possible to use latest Rust here and get rid of the problematic call_indirect instruction encoding afterwards in some way. This Rust issue about the topic suggests running through wasm2wat and wat2wasm, but that doesn't seem to be a lossless translation. For example, we use a custom section in the wasm for the migrate_version feature, which doesn't seem to be preserved in the text format.
So this would probably require a more targeted approach.

edit: Actually, it might be possible to disable reference types using some compiler flag? That would be much easier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants