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: [release-0.15] os to runner_os #905

Merged
merged 1 commit into from
Dec 25, 2024

Conversation

Hiroshiba
Copy link
Member

@Hiroshiba Hiroshiba commented Dec 25, 2024

内容

$OSとなってますが、おそらく正解は$RUNNER_OSです。
release-0.15でビルドするとなぜかcbindgenがインストールされておらず、いろいろ調べたところここじゃないかとなりました。
よくよく確認すると、この前macos-12をmacos-13に上げるrelease-0.15へのPRのGithub Workflowでもそこで落ちてました。

2箇所変更していますが、.github/actions/cargo-binstall-cbindgen/action.yml側は現在のmainブランチには存在してません。

.github/workflows/build_and_deploy.yml側の.exeを付けるか判定はこれでより正しくなるはず。
でも$OSはWindowsでだけWindows_NTと値が入ってるので、switch caseの仕様によっては正しく動作してるかも。
試してみた結果がこちら

その他

なんで以前はちゃんとOS判定できてた?のか不明。
判定はできてなくて、cbindgenがなぜか入ってたのか、あるいはキャッシュがヒットしてたとか・・・?

@Hiroshiba
Copy link
Member Author

ちょっと急いでいるのでマージします! 🙇
少なくともエラーになってたbuild_and_deploy.ymlは通りました。
release-0.15は再度mainブランチにマージすべきかも。

なぜかmacos-13とmacos-14のテストがすごい時間かかってそうで、マージ時点ではまだテストが回ってそうでした。
open_jtalk付近・・・?

test engine::open_jtalk::tests::extract_fullcontext_loop_works::case_1 has been running for over 60 seconds
test engine::open_jtalk::tests::extract_fullcontext_works::case_1 has been running for over 60 seconds
test engine::open_jtalk::tests::extract_fullcontext_works::case_2 has been running for over 60 seconds
test engine::synthesis_engine::tests::create_accent_phrases_works has been running for over 60 seconds

@Hiroshiba Hiroshiba changed the title add: [release-0.15] os to runner_os fix: [release-0.15] os to runner_os Dec 25, 2024
@Hiroshiba Hiroshiba merged commit f447189 into VOICEVOX:release-0.15 Dec 25, 2024
33 of 37 checks passed
@Hiroshiba Hiroshiba deleted the os-to-runner-os branch December 25, 2024 21:25
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.

1 participant