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

appveyor の成果物の zip 圧縮を検討する #109

Closed
m-tmatma opened this issue Jun 12, 2018 · 5 comments
Closed

appveyor の成果物の zip 圧縮を検討する #109

m-tmatma opened this issue Jun 12, 2018 · 5 comments
Labels
CI appveyor など CI 関連 【ChangeLog除外】
Milestone

Comments

@m-tmatma
Copy link
Member

https://www.appveyor.com/docs/packaging-artifacts/#artifacts-retention-policy
で appveyor の Artifacts が 6ヶ月の保存期間を過ぎたら削除されるポリシーが公開された。
(2018/6/17 から有効)

→ 多分 appveyor のディスク容量を圧迫しているんではないかと想像する。

オープンソースプロジェクトに対してタダで提供していただいていることに感謝しているので
appveyor のディスク容量の負担を減らすために zip 圧縮を検討する。

appveyor 上で 7z コマンドが利用可能なみたいで、7z コマンドを使えば zip 圧縮できる模様。
https://www.appveyor.com/docs/packaging-artifacts/#packaging-multiple-files-in-different-locations-into-a-single-archive

@m-tmatma m-tmatma added the CI appveyor など CI 関連 【ChangeLog除外】 label Jun 12, 2018
@m-tmatma
Copy link
Member Author

になみに zip 形式よりも 7z 形式のほうが圧縮率が高い。

https://sevenzip.osdn.jp/ で インストーラをダウンロードしてインストールするだけだけど
標準で対応していないのですこしユーザーのハードルは上がる。

CI から直でダウンロードしようとしているユーザーには問題ないかもしれないが。

@kobake
Copy link
Member

kobake commented Jun 13, 2018

うーん、確かにフリープランには感謝しているのですけど、僕らみたいな1プロジェクトがディスク圧迫を気にしても仕方ないような。
利用者目線での利便性を上げるために圧縮するのであれば賛成ですが、そうでなければそこにコストを払うのはどうだろう、という気持ちです。

自分が AppVeyor の中の人だとしたら、成果物は内部的には全部自動圧縮しておいて、ダウンロード時にだけ展開するように組みますね。そういうふうに全体のパフォーマンスチューニングするのであればコストと効果が見合う可能性はあると思っています。

@m-tmatma
Copy link
Member Author

#114 で 成果物を zip 圧縮を行うようにした。

対応のメリット

  • ダウンロード時間の短縮
  • ダウンロードした成果物の exe を実行したときに Smart Screen でブロックされなくなる(みたい)

成果物にビルド番号につけるというアイデアがあるが保留中

@kobake
Copy link
Member

kobake commented Jun 15, 2018

既存の有名なプロダクトがどのような方針を採っているのか気になってます

@m-tmatma
Copy link
Member Author

成果物にビルド番号につけるというアイデアがあるが保留中

#165 で登録した。

@m-tmatma m-tmatma added this to the next release milestone Jun 23, 2018
@ds14050 ds14050 added the CI appveyor など CI 関連 【ChangeLog除外】 label Sep 18, 2018
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this issue Jun 11, 2019
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this issue Jun 11, 2019
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this issue Jun 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI appveyor など CI 関連 【ChangeLog除外】
Projects
None yet
Development

No branches or pull requests

3 participants