test_utilクレートをdev-dependenciesに

[qryxip]

内容

Python APIからのtest_utilクレートの依存を、他に合わせてdependenciesからdev-dependenciesに移します。

https://penguin.fanbox.cc/posts/6211928

関連 Issue

その他

[Hiroshiba]

LGTM！！

今回はmanylinuxで必要になるライブラリがdev onlyだった感じですが、今後はちょっとした制約になるかもですねぇ。

[qryxip]

あ! なるほど...

https://github.com/VOICEVOX/voicevox_core/actions/runs/6655691778/job/18086507610?pr=659

[qryxip]

どうするべきか悩みますね... Java APIの方も、たまたまcargo buildが必要だからtest_utilのビルドを通る形になってますし。

voicevox_core/.github/workflows/test.yml

Lines 305 to 308 in dc8c3db

	- name: Build
	run: |
	cargo build -p voicevox_core_java_api -vv
	cargo build -p test_util -vv

[qryxip]

あ、これ2行目でtest_utilを明示的に対象にしているのか。ならPythonでも同様にすればいい...?

[Hiroshiba]

ちゃんとわかってないのですが、それに賛成です！！

[qryxip]

voicevox_core/.github/workflows/test.yml

Line 277 in dc8c3db

- run: cargo build -p voicevox_core_c_api -vv

ここ、#239のときに入れたのですが、当時の状態を見ても何故これが必要だったのか正確に思い出せない... MaturinがmacOSとかで変なことになったような思い出があるのでそれか...?

とりあえずここを"voicevox_core_c_api" → "test_util"に変えてみます。

[qryxip]

通った...
https://github.com/VOICEVOX/voicevox_core/actions/runs/6659635118?pr=659

ところでこの現状だと、dev-dependenciesに入れても意味をほとんど成しませんね。