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

自動ビルド: リリースをtgz形式に変更・ルートディレクトリを追加 #2

Merged
merged 3 commits into from
Dec 31, 2021

Conversation

aoirint
Copy link
Member

@aoirint aoirint commented Dec 31, 2021

内容

上のコメントの内容を反映させるPRです。

公式ReleaseではLinux用のビルドはtgz形式で配布されているので、Releaseにアップロードされる形式をzipからtgzに変更します。
Windows用のビルドが追加されることがあれば、分岐してzipにする形がいいかなと思っています。

また、公式Releaseでは圧縮ファイル内の一番上に、圧縮ファイル名と同名のディレクトリがありますが、このリポジトリのReleaseにはありません。コアのビルドは公式Releaseの構造を前提にしているので、構造を合わせます。

関連Issue

その他

改行コードがCRLFになっているのと、空行が消えているので調整したいです。

Rearchive artifactUpload to Releaseは、Release以外のときにJobが実行されることを想定していないですが、workflow_dispatchがあるので条件分岐を入れておいた方がよさそうだなと思いました(動いてそうなのでそのままでもいいかもですが)。

@aoirint aoirint marked this pull request as ready for review December 31, 2021 07:45
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!!

@Hiroshiba Hiroshiba merged commit f6e27ed into VOICEVOX:main Dec 31, 2021
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