Download LLVM only once for release builds

jacob-carlborg · Apr 17, 2019 · 39cceb0 · 39cceb0
1 parent c1657dc
commit 39cceb0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -5,9 +5,11 @@ d:
   - ldc
 
 install: ./tools/download_llvm.d
+
 before_script:
   - git fetch --unshallow
   - ./configure --llvm-path tmp/clang
+
 script: dub test --verror --compiler=${DC}
 
 os:
@@ -41,6 +43,7 @@ matrix:
       d: ldc
       os: linux
       env: LLVM_VERSION=8.0.0
+      install: ./tools/download_llvm.d --dstep
       script: ./tools/build_release.sh
       deploy:
         provider: releases

diff --git a/tools/build_release.sh b/tools/build_release.sh
@@ -6,10 +6,6 @@ app_name="dstep"
 target_dir="bin"
 target_path="$target_dir/$app_name"
 
-function download_llvm {
-  ./tools/download_llvm.d --dstep
-}
-
 function configure {
   local os=$(os)
   local default_flags="--llvm-path tmp/clang"
@@ -62,7 +58,6 @@ function archive {
   tar Jcf "$(release_name)".tar.xz -C "$target_dir" "$app_name"
 }
 
-download_llvm
 configure
 build
 test_dstep