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

Windows用実行バイナリの自動ビルド (#219) #264

Merged
merged 63 commits into from
Sep 29, 2021

Conversation

aoirint
Copy link
Member

@aoirint aoirint commented Sep 24, 2021

内容

Windows用実行バイナリの自動ビルドを行うWorkflowを追加します。

Artifact

  • windows-cpu-prepackage: zip形式 CPU版
  • windows-nvidia-prepackage: zip形式 CPU/GPU版
  • windows-nvidia: インストーラ形式 CPU/GPU版(分割7z、インストーラexeをまとめたzip)

関連 Issue

close #219

議論

electron:buildelectron:build --publish never

GitHub Releaseの自動作成機能がある? #235 とも関連していそう。よくわからない

VOICEVOXソフトウェアにおいて、CPU版・GPU版をそれぞれ配布するか?

@aoirint aoirint changed the title Windows・Linux用実行バイナリの自動ビルド (#219, #218) Windows用実行バイナリの自動ビルド (#219) Sep 24, 2021
@aoirint aoirint closed this Sep 24, 2021
@aoirint aoirint deleted the patch-build-workflow branch September 24, 2021 21:39
@aoirint aoirint restored the patch-build-workflow branch September 24, 2021 21:39
@aoirint aoirint reopened this Sep 24, 2021
@aoirint
Copy link
Member Author

aoirint commented Sep 25, 2021

コンフィグを試してみました。

npm run electron:build

npm run electron:build -- --config electron-builder.yml

では、ログでは同じelectron-builder.ymlがロードされた旨が表示されましたが、
前者はvue.config.jsの設定が使われ、
後者はelectron-builder.ymlの設定のみが使われているような挙動になりました(実行前にdist_electronを削除)。

vue.config.jsに記述した一部の設定だけ上書きすることはできないのかな、と思いました(electron-builder.ymlvue.config.jsの設定をすべてコピーする必要がある)。

@Hiroshiba
Copy link
Member

vue.config.jsに記述した一部の設定だけ上書きすることはできない

うーーーーーーーーーーん、なるほど・・・。悲しいですね・・・。
Issue建ててみても良いかもですね。

@aoirint aoirint marked this pull request as draft September 28, 2021 19:36
@aoirint aoirint marked this pull request as ready for review September 29, 2021 08:23
@aoirint
Copy link
Member Author

aoirint commented Sep 29, 2021

licenses.json の生成自動化をしたのでreadyに戻します。

VOICEVOX_ENGINE_REPO_URL: "https://github.com/aoirint/voicevox_engine"
VOICEVOX_ENGINE_VERSION: 0.6.0-aoirint-1

の変更が必要で、ENGINE側のlicenses.txtの存在を要求するので、
Hiroshiba/voicevox_engine@9fe3ff7 以降のENGINEのGitHub Releaseが必要になっています

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

良いですね!!

新しくENGINEのreleaseを作ってみているので、レビュー終了までにreleaseができたら切り替えてからマージしようと思います!
https://github.com/Hiroshiba/voicevox_engine/releases/tag/check-2021-09-29

.github/workflows/build.yml Outdated Show resolved Hide resolved
.github/workflows/build.yml Show resolved Hide resolved
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

LGTM!! ENGINEのリビルドはまだなので、一旦マージしようと思います。

ビルドは本当に手間だったので、自動ビルドを整備していただけて本当に嬉しいです・・・!

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.

ビルドの自動化
2 participants