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

Open
1 of 3 tasks
Hiroshiba opened this issue Sep 7, 2024 · 3 comments
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
機能向上 状態:実装者募集 実装者を募集している状態
Projects
None yet
Development

No branches or pull requests

2 participants