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

Windowsでビルドする時、エンジンマニフェストのcommandrun.exeにした方が良い #1470

Closed
1 of 3 tasks
Hiroshiba opened this issue Sep 7, 2024 · 4 comments · Fixed by #1545
Closed
1 of 3 tasks
Labels

Comments

@Hiroshiba
Copy link
Member

Hiroshiba commented Sep 7, 2024

内容

エンジンマニフェスト内のcommandには、ビルドしたパッケージディレクトリ内から、エンジンを起動するためのコマンド名を定義しています。
一般的にはエンジンファイルのパスを指します。

今のところVOICEVOXではcommand: "run"となっています。
しかし Windows のエンジンファイルのパスは正確にはrun.exeです。
ということで、Windows 版のビルドする時にエンジンマニフェスト内のcommandrun.exeにした方が良いのかなと思っています。

ターミナルとかでrunを起動しようとすれば勝手にrun.exeが起動するようになってるっぽいので、多分今のままでも問題はない・・・はず?
でも例えばrun.cmdがあったりするとややこしくなったり(優先されるのはexeっぽいけど)、やっぱりちゃんとrun.exeを指定する方が安心なのかなと。

Pros 良くなる点

分かりやすくなる

実現方法

パッケージのビルドは.github/workflows/build-engine-package.ymlで行っています。
このファイルのどこかで、windowsの場合はエンジンマニフェストを書き換えると良さそう。

VOICEVOXのバージョン

0.20.0

OSの種類/ディストリ/バージョン

  • Windows
  • macOS
  • Linux

その他

何か問題がありそうかどうかちょっと考えたいかも。

@Hiroshiba
Copy link
Member Author

@sabonerune いつも頼ってばかりですみません!
このissueの提案について不安なポイントとかあったら知りたいです 🙇

というか一度この辺り議論した記憶あるのですが、issue探しても見当たりませんでした。
放送内でちょっと話しただけ・・・?

@Hiroshiba Hiroshiba added the 要議論 実行する前に議論が必要そうなもの label Sep 7, 2024
@sabonerune
Copy link
Contributor

@Hiroshiba
Issueには書いた記憶はないのでDiscordか生放送内のどちらか(多分生放送)ですね。

.exeよりも優先順位が高いのは.comしかないからrun.exeと同じディレクトリにrun.comrunがなければ問題ないはず?
ただ実用上問題ないとはいえ正確な方がいいのではとは思います。

@Hiroshiba
Copy link
Member Author

Hiroshiba commented Sep 7, 2024

@sabonerune ありがとうございます!!

.com知りませんでした。ドメイン名でしか見たこと無かったです。

やったほうが良いだろうけど、優先度は高くないって感じですかね~~~。
ではとりあえず要議論は外して、実装者募集ステートに移したいと思います。


Github Actionsの勉強にちょうどいいくらいのタスク難度だと思います。
実装募集中です!

@Hiroshiba Hiroshiba added 状態:実装者募集 実装者を募集している状態 and removed 要議論 実行する前に議論が必要そうなもの labels Sep 7, 2024
@Hiroshiba Hiroshiba changed the title Windowsでビルドする時、エンジンマニフェストのcommandrun.exeにした方が良い? Windowsでビルドする時、エンジンマニフェストのcommandrun.exeにした方が良い Sep 7, 2024
Copy link

github-actions bot commented Mar 7, 2025

本 Issue は直近 180 日間で活動がありません。今後の方針について VOICEVOX チームによる再検討がおこなわれる予定です。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants