Skip to content

Conversation

@evilrobot-01
Copy link
Contributor

@evilrobot-01 evilrobot-01 commented Mar 26, 2025

Enables support for either wasm (default) or polkavm contracts. Tested for successful compilation only, not that tests pass.

The pop-contracts crate is split via v5 (default) and v6 features. The pop-cli crate is split into wasm-contracts (default) and polkavm-contracts features. I found being explicit much easier. The existing contract feature is retained as an alias to wasm-contracts. A new contracts feature alias wraps polkavm-contracts and would be the preferred option once wasm contracts are removed.

Default install (wasm contracts):

cargo install --path ./crates/pop-cli --locked

PolkaVM contracts only:

cargo install --path ./crates/pop-cli --locked --no-default-features -F polkavm-contracts

PolkaVM and parachain support:

cargo install --path ./crates/pop-cli --locked --no-default-features -F polkavm-contracts,parachain

@evilrobot-01 evilrobot-01 requested a review from AlexD10S March 26, 2025 14:40
Required for --no-default-features builds.
@AlexD10S AlexD10S force-pushed the feat/revive-compatibility branch 2 times, most recently from dbd4af0 to 24774c0 Compare March 27, 2025 08:56
@AlexD10S
Copy link
Collaborator

AlexD10S commented Apr 1, 2025

The content of this PR was merged into #500

@AlexD10S AlexD10S closed this Apr 1, 2025
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