CosmWasm / optimizer

Dockerfile and script to deterministically produce the smallest possible Wasm for your Rust contract
Apache License 2.0
120 stars 59 forks source link

Non x86_64 suffix #64

Closed maurolacy closed 2 years ago

maurolacy commented 2 years ago

Closes #63.

Add the "-arm64" suffix to Arm64 image names. Adds a suffix to non-Intel/Amd64 build artifacts, so that they can be distinguished from the official Intel/Amd64 built ones.

maurolacy commented 2 years ago

@webmaster128 do you agree with these changes?

webmaster128 commented 2 years ago

I was primarily thinking of suffixed in the docker images, not the artifacts. So cosmwasm/rust-optimizer:0.12.3 -> cosmwasm/rust-optimizer-arm64:0.12.3. Ideally we keep the Intel compile command as-is and do not require the additional platform parameter.

The suffix on the artifacts is a nice idea in addition to that.

maurolacy commented 2 years ago

Will do.