Skip to content

Commit c811b8a

Browse files
Address review by using env vars
1 parent d3766d8 commit c811b8a

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

.github/workflows/tail-call.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,10 @@ jobs:
8080
shell: cmd
8181
run: |
8282
choco install llvm --allow-downgrade --no-progress --version ${{ matrix.llvm }}.1.5
83-
./PCbuild/build.bat --tail-call-interp -d -p ${{ matrix.architecture }} "/p:PlatformToolset=clangcl" "/p:LLVMToolsVersion=19.1.5" "/p:LLVMInstallDir=C:\Program Files\LLVM"
83+
set PlatformToolset=clangcl
84+
set LLVMToolsVersion=19.1.5
85+
set LLVMInstallDir=C:\Program Files\LLVM
86+
./PCbuild/build.bat --tail-call-interp -d -p ${{ matrix.architecture }}
8487
./PCbuild/rt.bat -d -p ${{ matrix.architecture }} -q --multiprocess 0 --timeout 4500 --verbose2 --verbose3
8588
8689
# No tests (yet):
@@ -89,7 +92,10 @@ jobs:
8992
shell: cmd
9093
run: |
9194
choco install llvm --allow-downgrade --no-progress --version ${{ matrix.llvm }}.1.5
92-
./PCbuild/build.bat --tail-call-interp -p ${{ matrix.architecture }} "/p:PlatformToolset=clangcl" "/p:LLVMToolsVersion=19.1.5" "/p:LLVMInstallDir=C:\Program Files\LLVM"
95+
set PlatformToolset=clangcl
96+
set LLVMToolsVersion=19.1.5
97+
set LLVMInstallDir=C:\Program Files\LLVM
98+
./PCbuild/build.bat --tail-call-interp -p ${{ matrix.architecture }}
9399
94100
# The `find` line is required as a result of https://github.com/actions/runner-images/issues/9966.
95101
# This is a bug in the macOS runner image where the pre-installed Python is installed in the same

0 commit comments

Comments
 (0)