Rollback MSVC version

man-group · Nov 20, 2024 · 96eb298 · 96eb298
1 parent 8ee3763
commit 96eb298
Show file tree

Hide file tree

Showing 4 changed files with 25 additions and 12 deletions.
diff --git a/.github/workflows/benchmark_commits.yml b/.github/workflows/benchmark_commits.yml
@@ -18,7 +18,7 @@ jobs:
     needs: [start_ec2_runner]
     if: |
       always() &&
-      !cancelled()
+      !cancelled() && false
     runs-on: ${{ needs.start_ec2_runner.outputs.label }}
     container: ${{ inputs.cibw_image_tag}}
     env:

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -100,7 +100,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        python3: ${{fromJson(vars.LINUX_PYTHON_VERSIONS || '[6, 7, 8, 9, 10, 11]')}}
+        python3: ${{fromJson(vars.LINUX_PYTHON_VERSIONS || '[11]')}}
         arcticdb_version: ["oldest", "latest"]
         include:
           - python_deps_ids: [""]
@@ -139,6 +139,7 @@ jobs:
       python_deps_ids: ${{toJson(matrix.python_deps_ids)}}
 
   cpp-test-linux:
+    if: false
     needs: [cibw_docker_image, common_config]
     name: Linux C++ Tests
     uses: ./.github/workflows/build_steps.yml
@@ -150,12 +151,13 @@ jobs:
       matrix: ${{needs.common_config.outputs.linux_matrix}}
 
   build-python-wheels-linux:
+    if: false
     # Then use the cached compilation artifacts to build other python versions concurrently in cibuildwheels
     needs: [cibw_docker_image, common_config]
     strategy:
       fail-fast: false
       matrix:
-        python3: ${{fromJson(vars.LINUX_PYTHON_VERSIONS || '[6, 7, 8, 9, 10, 11]')}}
+        python3: ${{fromJson(vars.LINUX_PYTHON_VERSIONS || '[11]')}}
         include:
           - python_deps_ids: [""]
             matrix_override: ${{fromJson(needs.common_config.outputs.linux_matrix)}}
@@ -222,7 +224,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        python3: ${{fromJson(vars.LINUX_PYTHON_VERSIONS || '[6, 7, 8, 9, 10, 11]')}}
+        python3: ${{fromJson(vars.LINUX_PYTHON_VERSIONS || '[11]')}}
         arcticdb_version: ["oldest", "latest"]
         include:
           - python_deps_ids: [""]

diff --git a/.github/workflows/build_steps.yml b/.github/workflows/build_steps.yml
@@ -69,6 +69,18 @@ jobs:
     defaults:
       run: {shell: bash}
     steps:
+      - name: Download VS2022 BuildTools 17.11.5
+        if: matrix.os == 'windows'
+        run: |
+          choco install -y visualstudio2022buildtools --version=117.11.5 --params "--add Microsoft.VisualStudio.Component.VC.Tools.x86.x64 --installChannelUri https://aka.ms/vs/17/release/150476627_138095015/channel"
+          choco install -y ninja
+
+      #- name: Enable Windows compiler commands
+      #  if: matrix.os == 'windows'
+      #  uses: ilammy/[email protected]
+      #  with:
+      #    toolset: 14.41
+
       - name: Checkout
         uses: actions/[email protected]
         with:
@@ -81,16 +93,12 @@ jobs:
 
       - name: Windows Pagefile
         if: matrix.os == 'windows'
-        uses: al-cheb/configure-pagefile-action@v1.3
+        uses: al-cheb/configure-pagefile-action@v1.4
         with:
           minimum-size: 2GB
           maximum-size: 6GB
           disk-root: "D:"  # This is also the checkout directory. Total size 12GB.
         continue-on-error: true
-
-      - name: Enable Windows compiler commands
-        if: matrix.os == 'windows'
-        uses: ilammy/[email protected]
 
       - name: Extra envs
         # This has to come after msvc-dev-cmd to overwrite the bad VCPKG_ROOT it sets
@@ -123,13 +131,16 @@ jobs:
         run: . build_tooling/prep_cpp_build.sh # Also applies to Windows
 
       - name: CMake compile
-        if: inputs.job_type != 'build-python-wheels'
+        if: inputs.job_type != 'build-python-wheels' && matrix.os == 'windows'
+        env:
+          CC: "C:\\Program Files (x86)\\Microsoft Visual Studio\\2022\\BuildTools\\VC\\Tools\\MSVC\\14.41.34120\\bin\\Hostx64\\x64\\cl.exe"
+          CXX: "C:\\Program Files (x86)\\Microsoft Visual Studio\\2022\\BuildTools\\VC\\Tools\\MSVC\\14.41.34120\\bin\\Hostx64\\x64\\cl.exe"
         # We are pinning the version to 10.6 because >= 10.7, use node20 which is not supported in the container
         uses: lukka/[email protected]
         with:
           cmakeListsTxtPath: ${{github.workspace}}/cpp/CMakeLists.txt
           configurePreset: ${{env.ARCTIC_CMAKE_PRESET}}
-          configurePresetAdditionalArgs: "['-DVCPKG_INSTALL_OPTIONS=--clean-after-build']"
+          configurePresetAdditionalArgs: "['-DVCPKG_INSTALL_OPTIONS=--clean-after-build', '-DCMAKE_C_COMPILER=\"C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Tools\MSVC\14.41.34120\bin\Hostx64\x64\cl.exe\"', '-DCMAKE_CXX_COMPILER=\"C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Tools\MSVC\14.41.34120\bin\Hostx64\x64\cl.exe\"']"
           buildPreset: ${{env.ARCTIC_CMAKE_PRESET}}
 
       - name: Compile C++ tests

diff --git a/.github/workflows/build_with_conda.yml b/.github/workflows/build_with_conda.yml
@@ -23,7 +23,7 @@ jobs:
   linux:
     if: |
       always() &&
-      !cancelled()
+      !cancelled() && false
     runs-on: ubuntu-22.04
     env:
       ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION: true