Skip to content

Commit

Permalink
Merge pull request #133 from paulsengroup/fix/msvc
Browse files Browse the repository at this point in the history
Bring back compatibility with recent versions of MSVC
  • Loading branch information
robomics authored Nov 30, 2024
2 parents af46902 + dfb8f1b commit ceeeda8
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 11 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/pip.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ jobs:
strategy:
fail-fast: false
matrix:
os: [windows-2019, macos-latest, ubuntu-latest]
os: [windows-2022, macos-latest, ubuntu-latest]
python-version: ["3.9", "3.13"]

runs-on: ${{ matrix.os }}
Expand Down
10 changes: 0 additions & 10 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -159,14 +159,4 @@ if(WIN32)
)
endif()

if(MSVC)
if(MSVC_VERSION GREATER 1939)
message(
FATAL_ERROR
"MSVC versions after 19.39.33523.0 (and at least up to 19.42.34321.1) are known to produce broken hictkpy wheels. Please try again using MSVC 19.39.33523.0 (Visual Studio 17.9) or older."
)
endif()
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} /bigobj")
endif()

add_subdirectory(src)

0 comments on commit ceeeda8

Please sign in to comment.