background.tsでアプリの名称をvoicevoxとしてsetする

[Hiroshiba]

内容

VOICEVOXはCPU版とGPU版がある関係で、ビルド時にアプリの名称（productName）をやむなく分けています。

• CPU版自動ビルドを有効化 #306

これがアプリ固有のディレクトリの名称にも影響を与えていて、いくつかの問題を引き起こしています。

• CPU版のelectron-storeの保存先がvoicevox-cpuなのをGPU版と合わせたい #337
• mac版にlogsディレクトリが無い #1452

いっそのこともうapp.setNameでアプリの名称を動的に変えちゃうのもありかなと思いました。
background.ts内でアプリの名称を変更し、問題がないことを確認できたらこの課題は解決かなと。

Pros 良くなる点

CPU版とGPU版の名前の影響に怯えなくて済む

Cons 悪くなる点

ないはずだけど、問題が起こらないか検証がいろいろ必要

実現方法

background.tsの上の方でapp.setName("voicevox")的なことを書けばよさそう。
開発版とかは名称を分けた方が分かりやすいと思うので、suffixはつけた方が良さそう。

ただ設定ファイルの場所のマイグレーションコードもあるので要注意。
多分このbeforeUserDataDirのコードよりも下でsetNameすれば問題ない･･･？

voicevox/src/background.ts

Line 76 in 01729f4

const beforeUserDataDir = app.getPath("userData"); // 設定ファイルのマイグレーション用

その他

リポジトリをフォークした後Github Actionsでbuild.ymlをworkflow_dispatchすれば自動ビルドしてReleasesにアップロードされるので、色々問題なさそうかをそこでチェックできると思います。

[g-Ratie]

こちら取り組んでみます。

[Hiroshiba]

@g-Ratie コメントありがとうございます！！是非よろしくお願いします！！

[Hiroshiba]

取り組みありがとうございます！closeしました！