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

fix: dev版のアップロードが失敗する問題を修正 #1848

Merged
merged 3 commits into from
Feb 20, 2024

Conversation

sabonerune
Copy link
Contributor

内容

dmgのファイル名のスペースをピリオドに変更することで問題を修正します。
これによるリリース後のファイル名に影響はありません。

また、Windowsのインストーラーはビルド後にファイル名を置換してからアップロードしていましたが最初からスペースのないファイル名でビルドするように変更しました。

関連 Issue

その他

この処理はいらない?
https://github.com/sabonerune/voicevox/blob/02fbce2e7c7ccd323f6a7210b8da1094b162ca9f/.github/workflows/build.yml#L474-L479

@sabonerune sabonerune requested a review from a team as a code owner February 15, 2024 13:09
@sabonerune sabonerune requested review from Hiroshiba and removed request for a team February 15, 2024 13:10
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!!

ほんと助かります!!!🙇

その他に書かれてるとこ、mvせずそのままアップロードでよいのではってことですよね。
であれば不要に同感です!昔は1ディレクトリしか指定できなかったのかも(?)
ここもついでに変更お願いしても良いでしょうか。別PRや後回しでも大丈夫です!!

@sabonerune
Copy link
Contributor Author

修正しました。

PR作成時に忘れていたのですがmatrix.macos_executable_nameが未定義になっています。
ただ見たところ問題は起こってなさそう?
(mac持っていないので分からない)
https://github.com/sabonerune/voicevox/blob/992d4afab04075ec64f0e40389095ea321b5fb78/.github/workflows/build.yml#L297

@Hiroshiba
Copy link
Member

Hiroshiba commented Feb 18, 2024

PR作成時に忘れていたのですがmatrix.macos_executable_nameが未定義になっています。 ただ見たところ問題は起こってなさそう? (mac持っていないので分からない) sabonerune/voicevox@992d4af/.github/workflows/build.yml#L297

macos_executable_nameが空なのでディレクトリに実行権限をつけるコマンドになってそうですね!
普通ディレクトリには実行権限がついていると思うので、不要だと思います。ついでに消しちゃっていただければ!!

@sabonerune
Copy link
Contributor Author

修正しました。

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 ad6f682 into VOICEVOX:main Feb 20, 2024
9 checks passed
@sabonerune sabonerune deleted the fix/artifact-name-whitespace branch February 20, 2024 23:48
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.

softprops/action-gh-releaseがmacOSで同じファイルのアップロードに失敗する
2 participants