Conversation
| default: false | ||
| is_production: | ||
| description: "製品版をビルドする" | ||
| type: boolean | ||
| required: false | ||
| default: false |
There was a problem hiding this comment.
OSS版バイナリに署名することはないので二つのboolパラメータを統合して三値にした方がいいのではないかと思ったのですが、code_signingはもうcodesign.bashを実行するか否かに使われていないので、OSS版バイナリに署名しても事故ではないのならcode_signingとis_productionは排反でもいいと思います。
There was a problem hiding this comment.
ちょっと僕が勘違いしてるかもなのですが、code_signing引数でcodesign.bashを実行するかどうか制御してます。
https://github.com/Hiroshiba/voicevox_core/blob/45a9682b8d59fc99a634b2f27af516f7ba885d92/.github/workflows/build_and_deploy.yml#L239
production environmentを使うかどうかはproduction引数で制御してます。
production environmentを使うけど、コード署名はしない（実験版リリース）というケースを想定してます。
There was a problem hiding this comment.
気になったのはis_production && !code_signingのケースでした。ダミー音声版を署名したい場合があるのかなと思った次第です。ただcodesign.bash自体がActions上で動くかどうかを確かめたいというケースはありそうですね...
is_production |
code_signing |
|
|---|---|---|
false |
製品版を署名 (リリース) | ダミー音声版をビルドするだけ (普段のCI) |
true |
製品版を無署名 (お試しビルド) | ダミー音声版をビルドだけして、それに署名 (?) |
There was a problem hiding this comment.
署名するテストをしたいときは、今まで通りenvironmentを使わずにsecretsを設定すれば可能です。
（environmentはsecretsのサブセットを持っているだけ）
There was a problem hiding this comment.
なるほどこの状態でもできるんですね。
(↑の表のis_production、上下逆でした... あとis_production && !code_signingじゃなくて!is_production && code_signing...)
|
レビューありがとうございます！ マージしてビルドしてみます･･･！ |
内容
このリポジトリで製品ビルドを可能にします。
変更箇所は次のとおりです。
production環境を使う制御変数を用意するcode_signing環境の情報はproduction環境に統合しましたその他
実際にビルドしたものがこちらです。
これでかなり気軽にコアをビルドできるようになるはず！！