Skip to content

Commit

Permalink
AppVeyor: Skip saving cache on non-master branches
Browse files Browse the repository at this point in the history
Otherwise we run into situations where commits to stable branches
induce very long build times, as they have to basically build from
scratch but also invalidate the cache for future commits on the
master branch.

This commit also makes the cache folder branch-specific, but since
it's still limited to 1 GB of total cache size, we don't enable it
for non-master, as we would still run into issues with non-master
build invalidating the master cache.

(cherry picked from commit b021bdb)
  • Loading branch information
akien-mga committed Jul 3, 2019
1 parent bf54138 commit 332d74b
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions .appveyor.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@ environment:
TARGET: release_debug
ARCH: amd64

init:
- ps: if ($env:APPVEYOR_REPO_BRANCH -ne "master") { $env:APPVEYOR_CACHE_SKIP_SAVE = "true" }

cache:
- "%SCONS_CACHE_ROOT%"

Expand All @@ -26,7 +29,7 @@ before_build:
- python --version
- scons --version
- cl.exe
- SET "SCONS_CACHE=%SCONS_CACHE_ROOT%\2.1"
- set "SCONS_CACHE=%SCONS_CACHE_ROOT%\%APPVEYOR_REPO_BRANCH%"

build_script:
- scons platform=%GD_PLATFORM% target=%TARGET% tools=%TOOLS% verbose=yes progress=no
- scons platform=%GD_PLATFORM% target=%TARGET% tools=%TOOLS% verbose=yes progress=no

0 comments on commit 332d74b

Please sign in to comment.