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

0.14でconfig.jsonの場所が変わるのでマイグレーション #1094

Conversation

Hiroshiba
Copy link
Member

@Hiroshiba Hiroshiba commented Jan 7, 2023

内容

config.jsonの場所(設定の場所)が、以前はデバイス依存でディレクトリ名が変わっていたのが、0.14で%appdata%/voicevoxに固定されます。

このPRでは、以前の場所にconfig.jsonがあればコピーするようにします。

関連 Issue

close #1008

その他

開発バージョンの場合、以前は%appdata%/voicevox、今は%appdata%/voiecvox-devになっています。
なのでvoicevox/config.jsonを消したりvoiecvox-dev/config.jsonのバージョンを変えたりでデバッグできると思います。

あまり興味持たれない場所だと思うので、ある程度時間が経ってから見直されたときにわかるよう、ちょっと丁寧にログ出力しています。

@Hiroshiba Hiroshiba requested a review from a team as a code owner January 7, 2023 14:33
@Hiroshiba Hiroshiba requested review from y-chan and removed request for a team January 7, 2023 14:33
@Hiroshiba
Copy link
Member Author

もしよかったら @sabonerune さんもコメント頂けると嬉しいです・・・!!

@sabonerune
Copy link
Contributor

一通り動かしてみて動作自体は大丈夫だと思いました。
ただ、開発版の設定が存在しない場合0.14以降でも製品版からデータをコピーしてしまうので製品版の設定も消さないと開発版の設定がない場合のデバッグができないことが気になりました。

@Hiroshiba
Copy link
Member Author

レビューありがとうございます!

たしかに不便ですね・・・。(デバッグには便利だったのでそのままにしていました。)

ちょっと考えたのですが、npm runするときの引数とか.envの引数を見てconfigをclearするようにする、とかどうでしょう 👀
そもそも開発版のconfig.jsonを消しに行く作業が面倒だったことを思い出したので、結構楽になる・・・かも・・・?

@sabonerune
Copy link
Contributor

コピー元のバージョンが0.14以上だったらコピーしないとかで十分な気もします。

@Hiroshiba
Copy link
Member Author

ピー元のバージョンが0.14以上だったらコピーしないとかで十分な気もします。

良さそうです、追加します!!

@Hiroshiba
Copy link
Member Author

Hiroshiba commented Jan 8, 2023

@sabonerune 変更を書いてる途中に、開発版だったらマイグレーションしないようにすれば良いだけだと気づいたのでそうしてみました 🙇‍♂️

@Hiroshiba
Copy link
Member Author

@y-chan すみません、だいぶ小さいPRではあるのですが、なにか影響ありそうかだけ見て頂けると・・・!!

Copy link
Member

@y-chan y-chan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMです!

@Hiroshiba
Copy link
Member Author

ありがとうございます、マージします!

@Hiroshiba Hiroshiba merged commit cadd27e into VOICEVOX:main Jan 16, 2023
@Hiroshiba Hiroshiba deleted the config.yamlの場所のマイグレーション branch January 16, 2023 10: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.

設定ファイルの場所変更をマイグレーションする
3 participants