Skip to content

Conversation

@oech3
Copy link
Contributor

@oech3 oech3 commented Oct 9, 2025

内容

AppImageは圧縮済みなので、7zの圧縮レベルを0にして展開を早くします。
7z無しで単に分割する(#2750)とスクリプトの互換性が壊れる為。
残念ながら、この方法ではインストール中にディスク上に大きなファイルが出現するのまでは防げません(curlから7z -siにパイプできそうですが、できません)。

@voicevox-preview-pages
Copy link

🚀 プレビュー用ページを作成しました 🚀

更新時点でのコミットハッシュ:f1df9ce

@Hiroshiba
Copy link
Member

Hiroshiba commented Oct 10, 2025

@oech3
PRありがとうございます!
一応確認させていただけると!(可能なら両方とも実測値がほしいです・・・!)

  1. 容量はどれくらい増えますか?
  2. 展開速度はどれくらい早くなりますか?

@Hiroshiba Hiroshiba changed the title .AppImage.7zの圧縮レベルを0にする chore(build): .AppImage.7zの圧縮レベルを0にする Oct 10, 2025
@oech3
Copy link
Contributor Author

oech3 commented Oct 10, 2025

(同コミットで)/tmp上でtime 7z xをしてAppImage部分を出すまで。
変更前 2.52 sで展開, .7z.003は 686 MB
変更後 1.40 sで展開, .7z.003は 703 MB

@Hiroshiba Hiroshiba requested a review from Copilot October 11, 2025 04:19
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR optimizes the build process by setting the 7z compression level to 0 for AppImage files, since AppImage files are already compressed internally. This change aims to speed up the extraction process while maintaining compatibility with existing installation scripts.

  • Updated 7z command to use -mx=0 (no compression) when creating split archives
  • Added explanatory comments about why compression level 0 is used
  • Maintained the .7z format requirement for installer script compatibility

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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!!

調査ありがとうございます!

まあダウンロードサイズが20MB増えちゃうのはデメリットな気がしないでもないですが、2回圧縮する方が変な気がするのでこの形が良いのかな~と思いました。

一応 cc まで!! @sevenc-nanashi @sabonerune @madosuki

Copy link
Member

@sevenc-nanashi sevenc-nanashi left a comment

Choose a reason for hiding this comment

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

まぁ特に問題なさそう。

@Hiroshiba Hiroshiba enabled auto-merge October 11, 2025 04:29
@Hiroshiba Hiroshiba added this pull request to the merge queue Oct 11, 2025
Merged via the queue into VOICEVOX:main with commit 42e866a Oct 11, 2025
11 checks passed
@oech3 oech3 deleted the 7z-raw branch October 11, 2025 04:46
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.

3 participants