Skip to content

Optimize several wasm files at once. #2103

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.

Sign up for GitHub

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

Jump to bottom
Merged
fnando merged 1 commit into from
Jul 31, 2025
Merged

Optimize several wasm files at once. #2103

fnando merged 1 commit into from
Jul 31, 2025

Conversation

@fnando
Copy link
Member

@fnando fnando commented Jul 31, 2025

What

Make stellar contract optimize accept one or more files.

$ stellar contract optimize --wasm target/wasm32v1-none/release/hello_world.wasm target/wasm32v1-none/release/bye.wasm
Reading: target/wasm32v1-none/release/hello_world.wasm (574 bytes)
Optimized: target/wasm32v1-none/release/hello_world.optimized.wasm (537 bytes)
Reading: target/wasm32v1-none/release/bye.wasm (565 bytes)
Optimized: target/wasm32v1-none/release/bye.optimized.wasm (528 bytes)

$ stellar contract optimize
error: the following required arguments were not provided:
  --wasm <WASM>...

Usage: stellar contract optimize --wasm <WASM>...

For more information, try '--help'.

$ stellar contract optimize --wasm target/wasm32v1-none/release/hello_world.wasm target/wasm32v1-none/release/bye.wasm --wasm-out tmp/foo.wasm
❌ error: --wasm-out cannot be used with --wasm option when passing multiple files

Why

Close #717.

Known limitations

  • Cannot specify --wasm-out when passing more than 1 file via --wasm.

@github-project-automation github-project-automation bot moved this to Backlog (Not Ready) in DevX Jul 31, 2025
@fnando fnando merged commit c3a92be into main Jul 31, 2025
35 checks passed
@fnando fnando deleted the optimize-many branch July 31, 2025 21:06
@github-project-automation github-project-automation bot moved this from Backlog (Not Ready) to Done in DevX Jul 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@leighmcculloch leighmcculloch leighmcculloch approved these changes

@elizabethengelman elizabethengelman Awaiting requested review from elizabethengelman

Assignees

No one assigned

Labels

None yet

Projects

DevX
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Optimize should accept multiple wasm as arguments

3 participants

@fnando @leighmcculloch