Skip to content

Update preview2-prototyping download links#577

Merged
alexcrichton merged 2 commits intobytecodealliance:mainfrom
danbev:preview2-prototyping-download-links
Jun 28, 2023
Merged

Update preview2-prototyping download links#577
alexcrichton merged 2 commits intobytecodealliance:mainfrom
danbev:preview2-prototyping-download-links

Conversation

@danbev
Copy link
Copy Markdown
Contributor

@danbev danbev commented May 14, 2023

This commit updates the preview2-prototyping command and reactor .wasm download links as they have been updated.

This commit updates the preview2-prototyping command and reactor .wasm
download links as they have been updated.

Signed-off-by: Daniel Bevenius <daniel.bevenius@gmail.com>
@pchickey
Copy link
Copy Markdown
Contributor

Thanks! I would prefer to point these at the wasmtime repo once it lands there - I expect we will finish that tomorrow. bytecodealliance/wasmtime#6374

Comment thread README.md Outdated
Comment on lines +153 to +154
[non-command]: https://github.com/bytecodealliance/preview2-prototyping/releases/download/latest/wasi_preview1_component_adapter.reactor.wasm
[command]: https://github.com/bytecodealliance/preview2-prototyping/releases/download/latest/wasi_preview1_component_adapter.command.wasm
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[non-command]: https://github.com/bytecodealliance/preview2-prototyping/releases/download/latest/wasi_preview1_component_adapter.reactor.wasm
[command]: https://github.com/bytecodealliance/preview2-prototyping/releases/download/latest/wasi_preview1_component_adapter.command.wasm
[non-command]: https://github.com/bytecodealliance/wasmtime/releases/download/v10.0.1/wasi_snapshot_preview1.reactor.wasm
[command]: https://github.com/bytecodealliance/wasmtime/releases/download/v10.0.1/wasi_snapshot_preview1.command.wasm

Unfortunately, wasmtime doesn't have a "latest" tag, so we would have to update this on each wasmtime release, or we should open an issue on the wasmtime repo to add a latest tag.

Copy link
Copy Markdown
Contributor

@pchickey pchickey Jun 27, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe that the latest behavior is a github http redirect: https://github.com/bytecodealliance/wasmtime/releases/latest redirects to https://github.com/bytecodealliance/wasmtime/releases/tag/v10.0.1 at the moment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the suggestions! I've updated this PR now with a commit that uses latest like @pchickey suggested.

This commit updates the link to the component adapters to point to the
wasmtime repository instead of preview2-prototyping.

Signed-off-by: Daniel Bevenius <daniel.bevenius@gmail.com>
@alexcrichton alexcrichton merged commit 0cc1e99 into bytecodealliance:main Jun 28, 2023
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.

4 participants