appveyor の成果物の zip 圧縮する

[m-tmatma]

#109: appveyor の成果物を圧縮するようにする

対応のメリット

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

圧縮にかかる時間

• appveyor のログで確認する限り 2秒

call zipArtifacts.bat %PLATFORM% %CONFIGURATION% のログから
echo appveyor_yml までの時間で判断

[kobake]

3点指摘があります。

• zip ファイル名に sakura を含めていただきたいです。
  • 例： archive-Win32-Debug.zip → sakura-Win32-Debug.zip
• 可能であれば zip を解凍したときに Win32 のようなフォルダができるような形ではなく zip ファイル名に近い形のフォルダができるようになっていると解凍時にファイルが行方不明になったり混在しなくて助かります。
• zip に何のファイルが含められたのかがビルドログからパッと見で分かるようになっていると助かります。以下にビルドログ載せますが、変更前のログだと何が成果物であるかが一目でわかりますが、変更後のログだと zip を解凍するかビルドログ全読みしないといまいち成果物の内容物がわかりません。

変更前のビルド最下部ログ

....
....
Collecting artifacts...
Found artifact 'sakura\Debug\sakura.exe' matching 'sakura\Debug\sakura.exe' path
Found artifact 'sakura\Debug\sakura.pdb' matching 'sakura\Debug\sakura.pdb' path
Found artifact 'sakura\Release\sakura.exe' matching 'sakura\Release\sakura.exe' path
Found artifact 'sakura\Release\sakura.pdb' matching 'sakura\Release\sakura.pdb' path
Found artifact 'sakura_lang_en_US\Debug\sakura_lang_en_US.dll' matching 'sakura_lang_en_US\Debug\sakura_lang_en_US.dll' path
Found artifact 'sakura_lang_en_US\Release\sakura_lang_en_US.dll' matching 'sakura_lang_en_US\Release\sakura_lang_en_US.dll' path
Uploading artifacts...
[1/6] sakura\Debug\sakura.exe (5,683,200 bytes)...100%
[2/6] sakura\Debug\sakura.pdb (23,564,288 bytes)...100%
[3/6] sakura\Release\sakura.exe (1,612,288 bytes)...100%
[4/6] sakura\Release\sakura.pdb (17,215,488 bytes)...100%
[5/6] sakura_lang_en_US\Debug\sakura_lang_en_US.dll (177,664 bytes)...100%
[6/6] sakura_lang_en_US\Release\sakura_lang_en_US.dll (146,432 bytes)...100%
Build success

変更後のビルド最下部ログ

....
....
7-Zip [64] 16.04 : Copyright (c) 1999-2016 Igor Pavlov : 2016-10-04
Scanning the drive:
1 file, 5682688 bytes (5550 KiB)
Creating archive: archive-Win32-Debug.zip
Items to compress: 1
Files read from disk: 1
Archive size: 1632915 bytes (1595 KiB)
Everything is Ok
7-Zip [64] 16.04 : Copyright (c) 1999-2016 Igor Pavlov : 2016-10-04
Open archive: archive-Win32-Debug.zip
--
Path = archive-Win32-Debug.zip
Type = zip
Physical Size = 1632915
Scanning the drive:
1 file, 177664 bytes (174 KiB)
Updating archive: archive-Win32-Debug.zip
Items to compress: 1
Files read from disk: 1
Archive size: 1669393 bytes (1631 KiB)
Everything is Ok
7-Zip [64] 16.04 : Copyright (c) 1999-2016 Igor Pavlov : 2016-10-04
Open archive: archive-Win32-Debug.zip
--
Path = archive-Win32-Debug.zip
Type = zip
Physical Size = 1669393
Scanning the drive:
2 files, 23543808 bytes (23 MiB)
Updating archive: archive-Win32-Debug.zip
Items to compress: 2
Files read from disk: 2
Archive size: 6637502 bytes (6482 KiB)
Everything is Ok
echo appveyor_yml
appveyor_yml
Discovering tests...OK
Collecting artifacts...
Found artifact 'archive-Win32-Debug.zip' matching 'archive-Win32-Debug.zip' path
Uploading artifacts...
[1/1] archive-Win32-Debug.zip (6,637,502 bytes)...100%
Build success

[m-tmatma]

ローカルでの区別しやすさを考えると
ビルド番号を含めてもいいかも

デメリットはダウンロードしたzip に対して自動で何かしたいときに邪魔になること

[m-tmatma]

一応指摘には対応したつもりですが、
時間切れなのでまた後で

[kobake]

良い感じですね。成果物およびビルドログともに分かりやすくなったと思います。

--
Path = sakura-Win32-Debug.zip
Type = zip
Physical Size = 6634848
   Date      Time    Attr         Size   Compressed  Name
------------------- ----- ------------ ------------  ------------------------
2018-06-13 16:16:13 D....            0            0  sakura-Win32-Debug
2018-06-13 16:16:10 ....A      5682688      1632745  sakura-Win32-Debug\sakura.exe
2018-06-13 16:16:10 ....A     23457792      4963356  sakura-Win32-Debug\sakura.pdb
2018-06-13 16:16:12 ....A       177664        36293  sakura-Win32-Debug\sakura_lang_en_US.dll
2018-06-13 16:16:12 ....A        86016         1558  sakura-Win32-Debug\sakura_lang_en_US.pdb
------------------- ----- ------------ ------------  ------------------------
2018-06-13 16:16:13           29404160      6633952  4 files, 1 folders
echo appveyor_yml
appveyor_yml
Discovering tests...OK
Collecting artifacts...
Found artifact 'sakura-Win32-Debug.zip' matching 'sakura-Win32-Debug.zip' path
Uploading artifacts...
[1/1] sakura-Win32-Debug.zip (6,634,848 bytes)...100%
Build success

[kobake]

ビルド番号を含めてもいいかも

これの判断は難しいところです。一旦別件として検討しましょう。

時間切れなのでまた後で

今回はこの時点で内容問題ないと判断しましたのでマージしてしまいますが、
仮にマージされたくない場合には PR タイトルに WIP 付けてもらえればレビューとマージは保留します。

[kobake]

LGTM

[m-tmatma]

これの判断は難しいところです。一旦別件として検討しましょう。

#165 で登録した。