Skip to content

Commit

Permalink
nix: skip the sslyze test on aarch64 (#4050)
Browse files Browse the repository at this point in the history
  • Loading branch information
dougch committed Jun 27, 2023
1 parent 8893bc2 commit 5d7925f
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 12 deletions.
6 changes: 6 additions & 0 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -593,6 +593,12 @@ if (BUILD_TESTING)
# For Nix and environments where LD_LIBRARY_PATH is already correct.
# We're also dropping tox and calling pytest directly, because
# Nix is already handling all of the python setup.
if (CMAKE_SYSTEM_PROCESSOR STREQUAL "aarch64" AND ${test_target} STREQUAL "integrationv2_sslyze" )
# sslyze/nassl is not available on aarch64.
message(WARNING "Skipping ${test_target} due to missing tools on ${CMAKE_SYSTEM_PROCESSOR}")
continue()
endif()
message(STATUS "Adding integ test ${test_target}")
add_test(NAME ${test_target}
COMMAND
pytest
Expand Down
27 changes: 15 additions & 12 deletions nix/pyenv.nix
Original file line number Diff line number Diff line change
Expand Up @@ -33,15 +33,18 @@ let
pydantic # >=1.7,<1.9 TODO: check if this version is correct/if it matters
];
};
in pkgs.python310.withPackages (ps: [
ps.pep8
ps.pytest # ==5.3.5 TODO: check if this version is correct/if it matters
ps.pytest-xdist # ==1.34.0 TODO: check if this version is correct/if it matters
sslyze # ==5.0.2 TODO: check if this version is correct/if it matters
ps.pytest-rerunfailures
ps.tox
ps.typing-extensions
ps.setuptools-rust
ps.cryptography
nassl
])
in pkgs.python310.withPackages (ps:
[
ps.pep8
ps.pytest
ps.pytest-xdist
ps.pytest-rerunfailures
ps.typing-extensions
ps.setuptools-rust
ps.cryptography
] ++ (if pkgs.stdenv.hostPlatform.system == "x86_64-linux" then [
nassl
sslyze
] else
[ ]))

0 comments on commit 5d7925f

Please sign in to comment.