2323 config : Release
2424 runs-on : windows-latest
2525 steps :
26- - uses : actions/checkout@v3
27-
28- - id : setup-llvm
29- name : Set up LLVM (MSVC)
30- uses : ./.github/actions/setup-llvm-msvc
31- if : matrix.compiler == 'clang-cl'
26+ - uses : actions/checkout@v4
3227
3328 - name : Download nuget
3429 run : |
4944 $target_version = "1.2.3.4"
5045 $props = "Configuration=$target_configuration,Platform=$target_platform,CppWinRTBuildVersion=$target_version"
5146 if ("${{ matrix.compiler }}" -eq "clang-cl") {
52- $props += ",Clang=1,PlatformToolset=LLVM_v143,LLVMInstallDir=${{ steps.setup-llvm.outputs.llvm-path }} "
47+ $props += ",Clang=1,PlatformToolset=ClangCl "
5348 }
5449 Add-Content $env:GITHUB_ENV "msbuild_config_props=/p:$props"
5550
7166 cmd /c "$env:VSDevCmd" "&" msbuild /m /clp:ForceConsoleColor "$env:msbuild_config_props" cppwinrt.sln /t:cppwinrt
7267
7368 - name : Upload built executables
74- uses : actions/upload-artifact@v3
69+ uses : actions/upload-artifact@v4
7570 with :
76- name : msvc-build-${{ matrix.arch }}-${{ matrix.config }}-bin
71+ name : msvc-build-${{ matrix.compiler}}-${{ matrix. arch }}-${{ matrix.config }}-bin
7772 path : |
7873 _build/${{ matrix.arch }}/${{ matrix.config }}/*.exe
7974 _build/${{ matrix.arch }}/${{ matrix.config }}/*.dll
@@ -107,25 +102,20 @@ jobs:
107102 config : Release
108103 runs-on : windows-latest
109104 steps :
110- - uses : actions/checkout@v3
111-
112- - id : setup-llvm
113- name : Set up LLVM (MSVC)
114- uses : ./.github/actions/setup-llvm-msvc
115- if : matrix.compiler == 'clang-cl'
105+ - uses : actions/checkout@v4
116106
117107 - name : Fetch cppwinrt executables
118108 if : matrix.arch != 'arm64'
119- uses : actions/download-artifact@v3
109+ uses : actions/download-artifact@v4
120110 with :
121- name : msvc-build-${{ matrix.arch }}-${{ matrix.config }}-bin
111+ name : msvc-build-${{ matrix.compiler}}-${{ matrix. arch }}-${{ matrix.config }}-bin
122112 path : _build/${{ matrix.arch }}/${{ matrix.config }}/
123113
124114 - name : Fetch x86 cppwinrt executables (arm64 only)
125115 if : matrix.arch == 'arm64'
126- uses : actions/download-artifact@v3
116+ uses : actions/download-artifact@v4
127117 with :
128- name : msvc-build-x86-Release-bin
118+ name : msvc-build-${{ matrix.compiler}}- x86-Release-bin
129119 path : _build/x86/Release/
130120
131121 - name : Download nuget
@@ -147,7 +137,7 @@ jobs:
147137 $target_version = "1.2.3.4"
148138 $props = "Configuration=$target_configuration,Platform=$target_platform,CppWinRTBuildVersion=$target_version"
149139 if ("${{ matrix.compiler }}" -eq "clang-cl") {
150- $props += ",Clang=1,PlatformToolset=LLVM_v143,LLVMInstallDir=${{ steps.setup-llvm.outputs.llvm-path }} "
140+ $props += ",Clang=1,PlatformToolset=ClangCl "
151141 }
152142 Add-Content $env:GITHUB_ENV "msbuild_config_props=/p:$props"
153143
@@ -240,9 +230,9 @@ jobs:
240230
241231 - name : Upload arm64 test executables
242232 if : matrix.arch == 'arm64'
243- uses : actions/upload-artifact@v3
233+ uses : actions/upload-artifact@v4
244234 with :
245- name : msvc-tests-${{ matrix.arch }}-${{ matrix.config }}-bin
235+ name : msvc-tests-${{ matrix.test_exe }}-${{ matrix.compiler }}-${{ matrix. arch }}-${{ matrix.config }}-bin
246236 path : |
247237 _build/${{ matrix.arch }}/${{ matrix.config }}/*.exe
248238 _build/${{ matrix.arch }}/${{ matrix.config }}/*.dll
@@ -260,7 +250,7 @@ jobs:
260250 CMAKE_COLOR_DIAGNOSTICS : 1
261251 CLICOLOR_FORCE : 1
262252 steps :
263- - uses : actions/checkout@v3
253+ - uses : actions/checkout@v4
264254
265255 - name : Install cross compiler
266256 run : |
@@ -279,7 +269,7 @@ jobs:
279269 cmake --build build/cross_x64/ --target install -j2
280270
281271 - name : Upload cppwinrt.exe
282- uses : actions/upload-artifact@v3
272+ uses : actions/upload-artifact@v4
283273 with :
284274 name : cross-build-${{ matrix.arch }}-bin
285275 path : install/bin/cppwinrt.exe
@@ -293,7 +283,7 @@ jobs:
293283 Deployment : [Component, Standalone]
294284 runs-on : windows-latest
295285 steps :
296- - uses : actions/checkout@v3
286+ - uses : actions/checkout@v4
297287
298288 - name : Download nuget
299289 run : |
@@ -323,20 +313,22 @@ jobs:
323313 cmd /c "$env:VSDevCmd" "&" msbuild /m /clp:ForceConsoleColor "$env:msbuild_config_props" /p:Deployment=${{ matrix.Deployment }} natvis\cppwinrtvisualizer.sln
324314
325315 build-msvc-nuget-test :
326- name : ' Build nuget test'
316+ name : ' Build nuget test (${{ matrix.arch }}) '
327317 needs : test-msvc-cppwinrt-build
328318 strategy :
329319 matrix :
320+ compiler :
321+ - MSVC
330322 arch : [x86, x64]
331323 config : [Release]
332324 runs-on : windows-latest
333325 steps :
334- - uses : actions/checkout@v3
326+ - uses : actions/checkout@v4
335327
336328 - name : Fetch cppwinrt executables
337- uses : actions/download-artifact@v3
329+ uses : actions/download-artifact@v4
338330 with :
339- name : msvc-build-${{ matrix.arch }}-${{ matrix.config }}-bin
331+ name : msvc-build-${{ matrix.compiler}}-${{ matrix. arch }}-${{ matrix.config }}-bin
340332 path : _build/${{ matrix.arch }}/${{ matrix.config }}/
341333
342334 - name : Download nuget
@@ -371,17 +363,12 @@ jobs:
371363 - name : Run nuget test
372364 run : |
373365 cmd /c "$env:VSDevCmd" "&" msbuild /m /clp:ForceConsoleColor "$env:msbuild_config_props" test\nuget\NugetTest.sln
374- if ($LastExitCode -ne 0) {
375- echo "::warning::nuget test failed"
376- }
377- # FIXME: This build was failing from the start
378- exit 0
379366
380367 build-nuget :
381368 name : Build nuget package with MSVC
382369 runs-on : windows-latest
383370 steps :
384- - uses : actions/checkout@v3
371+ - uses : actions/checkout@v4
385372
386373 - name : Package
387374 run : |
@@ -396,7 +383,7 @@ jobs:
396383 }
397384
398385 - name : Upload nuget package artifact
399- uses : actions/upload-artifact@v3
386+ uses : actions/upload-artifact@v4
400387 with :
401388 name : package
402389 path : " *.nupkg"
0 commit comments