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

macosの.vvppパッケージがなぜか.vvpppになっている #1174

Closed
Hiroshiba opened this issue Apr 6, 2024 · 2 comments · Fixed by #1453
Closed

macosの.vvppパッケージがなぜか.vvpppになっている #1174

Hiroshiba opened this issue Apr 6, 2024 · 2 comments · Fixed by #1453
Labels
OS 依存:mac macOS に依存した現象 バグ 初心者歓迎タスク 初心者にも優しい簡単めなタスク 状態:実装者募集 実装者を募集している状態

Comments

@Hiroshiba
Copy link
Member

不具合の内容

エディター用のvvppパッケージは、ファイル容量が大きい場合を想定して.vvpppに分割することができるようになっています。
それがmacosは容量が小さいのでvvppになるはずですが、なってないことに気づきました。

macosの0.18.1のvvpp系パッケージ
image

現象・ログ

リリースビルドの中のmac用のvvppがvvpppになっている

再現手順

多分フォーク先でもgithub actionsのworkflow dispatchでビルドすれば同じようになるはず

期待動作

vvppになっている

VOICEVOXのバージョン

0.18.1

その他

そもそもエンジンのビルド時は自動的に分割してまずvvpppを作成します。
その後vvpppファイルが1つだけだった場合は自動的にvvppにリネームするようになっているはずです。
そこがなぜかmacだけうまく動いてないっぽい・・・?

この辺りのコードです

- name: Create VVPP archives
run: |
# Compress to compressed.zip.001, compressed.zip.002, ...
# NOTE: 1000th archive will be "compressed.zip.1000" after "compressed.zip.999". This is unconsidered as an extreme case.
(cd "${{ matrix.target }}" && 7z -r -v1900M a "../compressed.zip")
# Rename to artifact.001.vvppp, artifact.002.vvppp, ...
for FILE in $(ls "compressed.zip."*); do
NUMBER=${FILE##*.} # 001
mv "${FILE}" "${{ steps.vars.outputs.package_name }}.${NUMBER}.vvppp"
done
# Rename to artifact.vvpp if there are only artifact.001.vvppp
if [ "$(ls ${{ steps.vars.outputs.package_name }}.*.vvppp | wc -l)" == 1 ]; then
mv ${{ steps.vars.outputs.package_name }}.001.vvppp ${{ steps.vars.outputs.package_name }}.vvpp
fi
# Output splitted archive list
ls ${{ steps.vars.outputs.package_name }}*.vvppp ${{ steps.vars.outputs.package_name }}.vvpp > archives_vvpp.txt || true
mv archives_vvpp.txt "${{ steps.vars.outputs.package_name }}.vvpp.txt"

該当ソースコードのstepの上の方でset -euxして実行すれば、どういうコマンドが実行されているかログに出てくるはず・・・。
これで何かわかるかも・・・?

@Hiroshiba Hiroshiba added バグ OS 依存:mac macOS に依存した現象 labels Apr 6, 2024
@tarepan tarepan added the 状態:設計 設計をおこなっている状態 label Apr 7, 2024
@sabonerune
Copy link
Contributor

sabonerune commented Apr 8, 2024

macOSのwc(BSD wc?)はwc -lの出力にスペースが入るらしいです。
ref: https://egashira.dev/blog/wc-cmd-on-macos

@tarepan tarepan added the 要議論 実行する前に議論が必要そうなもの label Apr 9, 2024
@Hiroshiba
Copy link
Member Author

なるほどです!!
ChatGPTくんに聞いたところ、更に後ろに | tr -d '[:space:]'とかをつけるとトリムしてくれるかもと教えてくれました!

ちょっとどなたか実際に試していただけると・・・!

@Hiroshiba Hiroshiba added 初心者歓迎タスク 初心者にも優しい簡単めなタスク 状態:実装者募集 実装者を募集している状態 and removed 要議論 実行する前に議論が必要そうなもの 状態:設計 設計をおこなっている状態 labels Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OS 依存:mac macOS に依存した現象 バグ 初心者歓迎タスク 初心者にも優しい簡単めなタスク 状態:実装者募集 実装者を募集している状態
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants