Skip to content

Commit

Permalink
Re-enable Windows builds on GitHub Actions (#778)
Browse files Browse the repository at this point in the history
Signed-off-by: Juan Cruz Viotti <[email protected]>
  • Loading branch information
jviotti authored Jun 7, 2024
1 parent d6587f0 commit 46846a4
Showing 1 changed file with 15 additions and 9 deletions.
24 changes: 15 additions & 9 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,15 +45,12 @@ jobs:
cxx: g++
type: shared
shell: sh

# TODO: Add back once the SEH sudden exceptions are gone
# See: https://github.com/actions/runner-images/issues/10004
# - os: windows-latest
# type: static
# shell: pwsh
# - os: windows-latest
# type: shared
# shell: pwsh
- os: windows-latest
type: static
shell: pwsh
- os: windows-latest
type: shared
shell: pwsh

# Sanitizers
- os: ubuntu-latest
Expand Down Expand Up @@ -118,6 +115,15 @@ jobs:
cmake --install ./build --prefix ./build/dist --config Release --verbose
--component sourcemeta_jsontoolkit_dev
- name: Copy in the correct VC runtime DLLs (workaround for actions/runner-images#10004)
if: runner.os == 'windows'
run: |
Copy-Item (Join-Path `
((Get-ChildItem -Directory `
-Path "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\VC\Redist\MSVC\14.*" |
Sort -Descending | Select-Object -First 1).FullName
) 'x64\Microsoft.VC143.CRT\*.dll') "build\bin\Release"
# Not every CTest version supports the --test-dir option. If such option
# is not recognized, `ctest` will successfully exit finding no tests.
# Better to be sure and `cd` all the time here.
Expand Down

0 comments on commit 46846a4

Please sign in to comment.