chore: voicevox_onnxruntimeではcache-build-resultをスキップ

[qryxip]

内容

関連 Issue

Refs: VOICEVOX/voicevox_project#24

スクリーンショット・動画など

その他

[Hiroshiba]

あーなるほどです、わかってきました！！

voicevox関係ない普通のonnxruntimeをビルドして、そのキャッシュをvoicevox_onnxruntimeにも使いまわしたい、みたいな感じでしょうか。

[qryxip]

いえ、voicevox_onnxruntimeではactions/cacheを一切使わないようにする感じです。

第三者がキャッシュを引きずり出す方法が無いか心配だったのもありますが、キャッシュは1リポジトリにつき10GBしか持てないので、どちらかというとそちらの節約をしたい感じです。

[qryxip]

#62 (comment)を見て、このPRであればenv.TARGET_LIBRARY == 'voicevox_onnxruntime' $\implies$ steps.cache-build-result.outputs.cache-hit != 'true'が言えることに気付いたため、cache-hitの確認を飛ばしてみました:
470dd50 (#64)

[Hiroshiba]

あ～なるほどです！
voicevox_onnxruntimeのときもcache-hitを見るコードに引っ張られて、意味を間違えて逆に捉えてしまっていました。

この方針なのであれば問題なさそう！！

[Hiroshiba]

LGTM！！

ぶっちゃけ通常onnxruntimeでも広い範囲で同じキーを使ったキャッシュはやめたほうが良いとは思いますが、間違っててもVOICEVOXにとって痛手ではないのもあり、ここが落とし所なのかなーと感じました。