Skip to content
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

wasmtime と wasmtime-dotnet のバージョンアップ: 16.0.0 #1

Merged
merged 4 commits into from
Jan 26, 2024

Conversation

Cj-bc
Copy link
Contributor

@Cj-bc Cj-bc commented Jan 21, 2024

wasmtimeとwasmtime-dotnetを、現状の最新リリースの16.0.0にアップデートしました。

動作確認環境

OS: Windows11
Unity: 2021.3.0f1, 2021.3.27f1

Assets/Mochineko/WasmtimeDotNetUnity/Demo/WasmDemoScene.unityを開いて実行し、ログメッセージとCube created from WebAssembly が生成されることを確認しました。他のOSでの動作確認は出来ていません。

変更点

Assets/BytecodeAlliance/WasmtimeDotNet/

https://github.com/bytecodealliance/wasmtime-dotnet/releases/tag/v16.0.0 からダウンロードし、更新しました。

独自修正箇所

1箇所C# 10から追加された機能が使われておりコンパイルエラーを吐かれたので修正しています(Cj-bc@fd56659 )。

Assets/BytecodeAlliance/Wasmtime/Plugins/

からライブラリを取得し、更新しました。

独自修正箇所

取得したライブラリが libwasmtime* と命名されていたのを、このレポジトリの元の状態に合わせて wasmtime* に変更しました。
公式のwasmtime ver5.0.0のリリースでも同様に libwasmtime* と命名されていたことから、名前を変更しても問題がないのかなと思い変更しています。

wasmtime.a はLinux用とmacOS用どちらからも提供されていましたが、f77a90e にてmacOS側のものを使用しているようだったのでそちらを入れています。

そのほか

bytecodealliance/wasmtime#7315 以降全ライブラリそれぞれに、対応した -min という名前のものが追加されています。
上記PRを見るに、これらは「デフォルトの機能がすべて無効化されたもの」であり「wasmtimeの大雑把な大きさを図るのに使いやすい」ので追加されているようでした。

his commit updates the binary artifacts produced by CI to include "min" builds where all default features are disabled

Furthermore the C API which previously had libwasmtime.so for example now has libwasmtime-min.so. The intention
is that the minimum-size artifacts are handy for determining a rough size of Wasmtime but they're not so important that it seems worthwhile
to dedicate entire release entries for.

そのため、これらは追加しなくて良いものかなと思っています。

Copy link
Owner

@mochi-neko mochi-neko left a comment

Choose a reason for hiding this comment

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

Pull requestありがとうございます、確認いたしました。

@mochi-neko mochi-neko merged commit 168d9a3 into mochi-neko:main Jan 26, 2024
@Cj-bc Cj-bc deleted the update-wasmtime-to-16.0.0 branch January 26, 2024 15:58
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.

2 participants