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

Fix opslang-wasm packaging #134

Merged
merged 1 commit into from
Apr 19, 2024
Merged

Fix opslang-wasm packaging #134

merged 1 commit into from
Apr 19, 2024

Conversation

sksat
Copy link
Member

@sksat sksat commented Apr 19, 2024

概要

opslang-wasm crate が cargo package できるようにする

変更の意図や背景

  • Release 0.7.0-beta.5 #132 の時点だと、opslang-wasm crate で cargo package すると Cargo.lock が(target/package 中の)ソースディレクトリ中に発生してしまい、エラーになってしまう
  • この問題の素朴な解決策としては opslang-wasm のソースディレクトリを(x86_64/aarch64 ビルドの)$OUT_DIR 内にコピーしてそのディレクトリ内で wasm-pack build することが考えられる
  • しかし(!)cargo package の時は Cargo.tomlversion.workspace = true などを解決済みの Cargo.toml がソースディレクトリ中に存在するが、通常の cargo build などの時についてはそうではない
    • そのため、cargo build の時は、version.workspace = true などが未解決のままの(workspace member であるはずの)ソースディレクトリ一式が $OUT_DIR に単独でコピーされ、ビルドに使われる
    • このビルドは workspace root のものとして走るので、version.workspace = true などを解決できずビルドに失敗してしまう
  • そこで(!!!)wasm-pack build の後、発生してしまったソースディレクトリ中の Cargo.lockbuild.rs 中で削除することにした
  • ついでに wasm-opslang のビルドロジックについてコメントを追加

発端となる Issue

@sksat sksat self-assigned this Apr 19, 2024
@sksat sksat requested a review from KOBA789 as a code owner April 19, 2024 10:46
@KOBA789 KOBA789 merged commit 6db716b into main Apr 19, 2024
4 checks passed
@KOBA789 KOBA789 deleted the fix-opslang-wasm-packaging branch April 19, 2024 10:48
@sksat sksat added this to the v0.7.0 milestone Apr 19, 2024
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