Java API：ビルドWorkflowを追加 #621
Conversation
[skip ci]
[skip ci]
[skip ci]
[skip ci]
[skip ci]
[skip ci]
[skip ci]
[skip ci]
[skip ci]
[skip ci]
[skip ci]
[skip ci]
[skip ci]
[skip ci]
| - name: Set up Java | ||
| uses: actions/setup-java@v2 | ||
| with: | ||
| java-version: "17" |
There was a problem hiding this comment.
Android Gradle Pluginは17以上じゃないと動かなかったので。
| - name: set cargo version | ||
| run: | | ||
| cargo set-version "$VERSION" --exclude voicevox_core_python_api --exclude download --exclude xtask | ||
| - name: "Download artifact (windows-x64-cpu)" |
There was a problem hiding this comment.
かなり混沌としてるので何とかしたい（何とかできなさそうな感じはする）
There was a problem hiding this comment.
github cli使ってダウンロードするところからgradle叩くところまでをシェルスクリプトにして実行、とかですかねぇ 😇
（本当はこっちのほうがローカルでも動かせるので良い）
There was a problem hiding this comment.
github cliは実行中のArtifactを落とせないっぽみがありました（記憶違いかも）
| cp -v "artifact/$artifact_name"/* "crates/voicevox_core_java_api/lib/src/main/resources/jniLibs/${target}/" | ||
| done | ||
|
|
||
| cp ${{ steps.setup-ndk.outputs.ndk-path }}/toolchains/llvm/prebuilt/linux-x86_64/sysroot/usr/lib/aarch64-linux-android/libc++_shared.so crates/voicevox_core_java_api/lib/src/main/resources/jniLibs/arm64-v8a/ |
There was a problem hiding this comment.
stdc++_shared.soがないとロード時にクラッシュするので。
| ls -l ./target/wheels | ||
| echo "whl=$(find ./target/wheels -type f | head -1)" >> "$GITHUB_OUTPUT" |
There was a problem hiding this comment.
分からないですね…なぜか落ちるようになりました
There was a problem hiding this comment.
もしかしたらキャッシュが原因かなと思いました。やっぱキャッシュむずい！！！！！！！！！！！！！
There was a problem hiding this comment.
あー！その可能性はありますね
上でrm -rf ./target/wheels/*を走らせれば直りはするでしょうが、まぁ怖くなってきました
| - name: cache target | ||
| uses: actions/cache@v3 | ||
| if: github.event.inputs.is_production != 'true' | ||
| with: | ||
| path: | | ||
| ~/.cargo | ||
| target | ||
| key: ${{ matrix.artifact_name }}-rust-${{ hashFiles('**/Cargo.lock') }} | ||
| restore-keys: | | ||
| ${{ matrix.artifact_name }}-rust- |
There was a problem hiding this comment.
結構早くなる感じでしょうか？
（キャッシュ、すぐバグにつながる印象があるのでちょっと避けたいなという思いがあります 😇 ）
There was a problem hiding this comment.
5分くらいですね。
ビルドにスピードが求められるのはWorkflowテストくらいなので製品版ビルドでは無効化しています
There was a problem hiding this comment.
なるほどです、確かに5分短縮は嬉しいのですが･･･。
キャッシュのキーってこのリポジトリ全てで共通なグローバル変数で、かつ未来方向と過去方向にもちゃんと一意になっているかを意識し続けないといけないため、相当気を使うんですよね･･･。
これ以降のコードがキャッシュファイルがあることを想定して書く必要もあり、かつそうなっていない気がするので、できれば避けたい気持ちがあります。
There was a problem hiding this comment.
https://github.com/Swatinem/rust-cacheではなくactions/cacheを使っている理由はあったりしますでしょうか?
(キーとしてはCargo.lock以外にもrust-toolchainだったりあるかなと思った次第で、rust-cacheはそれも含んでくれます)
There was a problem hiding this comment.
ないですね、そっちにしてみました。（まぁキャッシュ自体を消す可能性もありますが）
[skip ci]
[skip ci]
[skip ci]
Android版だけにしました。 |
Co-authored-by: Ryo Yamashita <[email protected]>
|
レビューを反映しました。 |
|
マージします。 |
内容
Java APIをビルドするWorkflowを追加します。
publishToMavenLocalした後に
~/.m2/repository/をzip圧縮しています。https://github.com/sevenc-nanashi/vv_core_registry/blob/c874a9569934d8442160353b6a2422fc4870cd5b/inject.ts#L166 こんな感じのスクリプトを書けば配信できます。
関連 Issue
その他
サンプルビルド： https://github.com/sevenc-nanashi/voicevox_core/releases/tag/0.15.0-preview.java-workflow.3