From 9b8f04bc55578dbc47b56841826d658e91a22b3a Mon Sep 17 00:00:00 2001 From: Peter Wu Date: Sun, 10 Feb 2019 18:30:50 +0100 Subject: [PATCH] appveyor: upload installer artifacts for development branches Manually configuring the artifacts at the configuration page does not work, so let's make artifact uploads automatic again. It was previously disabled because Dario triggers a lot of builds due to mirroring master. That issue is now resolved by limiting uploads to other branches. Change-Id: Id7522c1890ec749b73f9ee16ddbe76a363235663 Reviewed-on: https://code.wireshark.org/review/31962 Reviewed-by: Dario Lombardo Reviewed-by: Peter Wu --- appveyor.yml | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/appveyor.yml b/appveyor.yml index 349618086d..cbf4303be8 100644 --- a/appveyor.yml +++ b/appveyor.yml @@ -49,8 +49,12 @@ before_build: after_build: - msbuild /m nsis_package_prep.vcxproj - msbuild /m nsis_package.vcxproj - # Relocate the installer such that the artifact URL looks nicer. - - move packaging\nsis\*.exe .. + # Publish installers only for development branches to avoid exceeding the + # artifact storage limit of 50 GB. + - ps: | + if ($Env:APPVEYOR_REPO_BRANCH -notlike "master*") { + Get-Item packaging\nsis\Wireshark-*.exe | % { Push-AppveyorArtifact $_.FullName -FileName $_.Name } + } before_test: - msbuild /m test-programs.vcxproj @@ -63,11 +67,4 @@ on_finish: $Tshark = "run\\$Env:CONFIGURATION\\tshark.exe" if (Test-Path $Tshark) { & $Tshark --version } -# AppVeyor prevents to store artifacts exceeding 50 GB. Commenting out -# their upload for now. They can be re-enabled on your own repo from the -# configuration page: -# https://ci.appveyor.com/project//wireshark/settings/artifacts -# artifacts: -# - path: Wireshark-*.exe - deploy: off