-
Notifications
You must be signed in to change notification settings - Fork 205
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
コード署名する #430
The head ref may contain hidden characters: "\u30B3\u30FC\u30C9\u7F72\u540D"
コード署名する #430
Conversation
@aoirint さんや、 @PickledChair さん、 @y-chan さんもしよければレビューいただけると嬉しいです!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTMです
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
environment
の仕様・使い方についてよく知らないこともあり、うまく動くのかどうか自信が持てない部分があったので質問しました(あと typo を発見したのでコメントしました)。ご確認いただければと思います……!
Co-authored-by: Gray Suitcase <41382894+PickledChair@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
environment
の指定に関しては了解しました! 追加で変更が必要と思われる箇所を見つけたのでコメントしました。
Co-authored-by: Gray Suitcase <41382894+PickledChair@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
こちら、マージします!(コアの方の VOICEVOX/voicevox_core#167 の結果が大丈夫かどうか見守っているうちにマージし忘れていました。すみません……!) |
内容
run.exeをコード署名します。
証明書.pfxファイルをbase64エンコードしたものをsecret.CERT_BASE64に登録し、
証明書のパスワードをsecret.CERT_PASSWORDに登録して、
workflow_dispatchで「コード署名する」にチェックを入れてbuild.ymlを起動すれば、
core.dllが署名されます。
もしくはcode_signingenvironmentを作成し、同様にCERT_BASE64とCERT_PASSWORDを登録して実行しても署名できます。
(こちらのフローは、OSSリポジトリ上でビルドする場合を想定しています。)
「コード署名する」にチェックを入れずにビルドしたり、releaseを作ってビルドした場合は署名用のコードが実行されないので、メンテナー以外の方は今まで通りにテストビルドができます。
関連 Issue
スクリーンショット・動画など
その他
実際にこちらでビルドを試しています。
実際のビルド結果がこちらです。
https://github.com/Hiroshiba/voicevox_engine/releases/tag/check-code-sign