VOICEVOX / voicevox_engine

無料で使える中品質なテキスト読み上げソフトウェア、VOICEVOXの音声合成エンジン
https://voicevox.hiroshiba.jp/
Other
1.29k stars 195 forks source link

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

Open Hiroshiba opened 4 weeks ago

Hiroshiba commented 4 weeks ago

内容

エンジンマニフェスト内の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の種類/ディストリ/バージョン

その他

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

Hiroshiba commented 4 weeks ago

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

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

sabonerune commented 4 weeks ago

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

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

Hiroshiba commented 4 weeks ago

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

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

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


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