From dedda228f44a64d73afff120fd6e47b2a27170c2 Mon Sep 17 00:00:00 2001 From: yangliz5 Date: Wed, 26 Jan 2022 00:32:33 -0600 Subject: [PATCH 1/2] Fix a typo in class.rst (#3648) Fix a typo in class.rst --- docs/advanced/classes.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/advanced/classes.rst b/docs/advanced/classes.rst index 6330af5eb..ba006252a 100644 --- a/docs/advanced/classes.rst +++ b/docs/advanced/classes.rst @@ -9,7 +9,7 @@ that you are already familiar with the basics from :doc:`/classes`. Overriding virtual functions in Python ====================================== -Suppose that a C++ class or interface has a virtual function that we'd like to +Suppose that a C++ class or interface has a virtual function that we'd like to override from within Python (we'll focus on the class ``Animal``; ``Dog`` is given as a specific example of how one would do this with traditional C++ code). From bcb6d63ce99752dd735b5abf128f498458999a66 Mon Sep 17 00:00:00 2001 From: Liam Keegan Date: Wed, 26 Jan 2022 17:18:08 +0100 Subject: [PATCH 2/2] fix msys ci python issue (#3651) * msys ci: un-pin setup-msys2 action version * msys ci: explicitly set PYTHONHOME and PYTHONPATH for c++ and interface tests (to workaround https://github.com/msys2/setup-msys2/issues/167) --- .github/workflows/ci.yml | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 6e0ffa63a..38b85aeef 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -904,8 +904,7 @@ jobs: - { sys: mingw64, env: x86_64 } - { sys: mingw32, env: i686 } steps: - # Force version because of https://github.com/msys2/setup-msys2/issues/167 - - uses: msys2/setup-msys2@v2.4.2 + - uses: msys2/setup-msys2@v2 with: msystem: ${{matrix.sys}} install: >- @@ -935,10 +934,10 @@ jobs: run: cmake --build build --target pytest -j 2 - name: C++11 tests - run: cmake --build build --target cpptest -j 2 + run: PYTHONHOME=/${{matrix.sys}} PYTHONPATH=/${{matrix.sys}} cmake --build build --target cpptest -j 2 - name: Interface test C++11 - run: cmake --build build --target test_cmake_build + run: PYTHONHOME=/${{matrix.sys}} PYTHONPATH=/${{matrix.sys}} cmake --build build --target test_cmake_build - name: Clean directory run: git clean -fdx @@ -953,10 +952,10 @@ jobs: run: cmake --build build2 --target pytest -j 2 - name: C++14 tests - run: cmake --build build2 --target cpptest -j 2 + run: PYTHONHOME=/${{matrix.sys}} PYTHONPATH=/${{matrix.sys}} cmake --build build2 --target cpptest -j 2 - name: Interface test C++14 - run: cmake --build build2 --target test_cmake_build + run: PYTHONHOME=/${{matrix.sys}} PYTHONPATH=/${{matrix.sys}} cmake --build build2 --target test_cmake_build - name: Clean directory run: git clean -fdx @@ -971,7 +970,7 @@ jobs: run: cmake --build build3 --target pytest -j 2 - name: C++17 tests - run: cmake --build build3 --target cpptest -j 2 + run: PYTHONHOME=/${{matrix.sys}} PYTHONPATH=/${{matrix.sys}} cmake --build build3 --target cpptest -j 2 - name: Interface test C++17 - run: cmake --build build3 --target test_cmake_build + run: PYTHONHOME=/${{matrix.sys}} PYTHONPATH=/${{matrix.sys}} cmake --build build3 --target test_cmake_build