Skip to content

Commit 0f4761b

Browse files
authored
Rollback of DOWNLOAD_CATCH=OFF change merged via PR #3059. (#3092)
1 parent cf006af commit 0f4761b

File tree

1 file changed

+13
-13
lines changed

1 file changed

+13
-13
lines changed

.github/workflows/ci.yml

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ jobs:
100100
run: >
101101
cmake -S . -B .
102102
-DPYBIND11_WERROR=ON
103-
-DDOWNLOAD_CATCH=OFF
103+
-DDOWNLOAD_CATCH=ON
104104
-DDOWNLOAD_EIGEN=ON
105105
-DCMAKE_CXX_STANDARD=11
106106
${{ matrix.args }}
@@ -111,10 +111,10 @@ jobs:
111111
- name: Python tests C++11
112112
run: cmake --build . --target pytest -j 2
113113

114-
#- name: C++11 tests
115-
# # TODO: Figure out how to load the DLL on Python 3.8+
116-
# if: "!(runner.os == 'Windows' && (matrix.python == 3.8 || matrix.python == 3.9 || matrix.python == '3.10-dev'))"
117-
# run: cmake --build . --target cpptest -j 2
114+
- name: C++11 tests
115+
# TODO: Figure out how to load the DLL on Python 3.8+
116+
if: "!(runner.os == 'Windows' && (matrix.python == 3.8 || matrix.python == 3.9 || matrix.python == '3.10-dev'))"
117+
run: cmake --build . --target cpptest -j 2
118118

119119
- name: Interface test C++11
120120
run: cmake --build . --target test_cmake_build
@@ -127,7 +127,7 @@ jobs:
127127
run: >
128128
cmake -S . -B build2
129129
-DPYBIND11_WERROR=ON
130-
-DDOWNLOAD_CATCH=OFF
130+
-DDOWNLOAD_CATCH=ON
131131
-DDOWNLOAD_EIGEN=ON
132132
-DCMAKE_CXX_STANDARD=17
133133
${{ matrix.args }}
@@ -139,10 +139,10 @@ jobs:
139139
- name: Python tests
140140
run: cmake --build build2 --target pytest
141141

142-
#- name: C++ tests
143-
# # TODO: Figure out how to load the DLL on Python 3.8+
144-
# if: "!(runner.os == 'Windows' && (matrix.python == 3.8 || matrix.python == 3.9 || matrix.python == '3.10-dev'))"
145-
# run: cmake --build build2 --target cpptest
142+
- name: C++ tests
143+
# TODO: Figure out how to load the DLL on Python 3.8+
144+
if: "!(runner.os == 'Windows' && (matrix.python == 3.8 || matrix.python == 3.9 || matrix.python == '3.10-dev'))"
145+
run: cmake --build build2 --target cpptest
146146

147147
- name: Interface test
148148
run: cmake --build build2 --target test_cmake_build
@@ -754,7 +754,7 @@ jobs:
754754
cmake -S . -B build
755755
-G "Visual Studio 16 2019" -A Win32
756756
-DPYBIND11_WERROR=ON
757-
-DDOWNLOAD_CATCH=OFF
757+
-DDOWNLOAD_CATCH=ON
758758
-DDOWNLOAD_EIGEN=ON
759759
${{ matrix.args }}
760760
- name: Build C++11
@@ -800,7 +800,7 @@ jobs:
800800
cmake -S . -B build
801801
-G "Visual Studio 14 2015" -A x64
802802
-DPYBIND11_WERROR=ON
803-
-DDOWNLOAD_CATCH=OFF
803+
-DDOWNLOAD_CATCH=ON
804804
-DDOWNLOAD_EIGEN=ON
805805
806806
- name: Build C++14
@@ -849,7 +849,7 @@ jobs:
849849
cmake -S . -B build
850850
-G "Visual Studio 15 2017" -A x64
851851
-DPYBIND11_WERROR=ON
852-
-DDOWNLOAD_CATCH=OFF
852+
-DDOWNLOAD_CATCH=ON
853853
-DDOWNLOAD_EIGEN=ON
854854
-DCMAKE_CXX_STANDARD=${{ matrix.std }}
855855
${{ matrix.args }}

0 commit comments

Comments
 (0)