8 Commits (ea46d8f240071540dae1926800a89f3c400e3441)

Author	SHA1	Message	Date
liushuyu	2cc3afe1aa	CI: update various GitHub Actions imports in cmake.yml	9 months ago
Micah Snyder	7913c21c86	GitHub Actions (Windows): Fix link issue ... The build is running a different link.exe than the MSVC linker, possibly the one provided by bash. Fix by deleting /usr/bin/link.exe See: https://yncat.github.io/2022/02/18/github-actions%E3%81%A7-msvc-%E3%81%AE-link-%E3%81%8C%E4%BD%BF%E3%81%88%E3%81%AA%E3%81%8F%E3%81%AA%E3%82%8B%E8%A9%B1.html	2 years ago
Micah Snyder	c34dd0fa26	GH Actions: Newer version of run-vcpkg, older version of Windows ... The 'windows-latest' image is now windows-2022, but the vcpkg we're using doesn't know to look for visual studio 2022. Reverting to use windows-2019 so vcpkg works with it. Also switching to the run-vcpkg v7.4. Not going with the new v10 series because that changes the inputs and I'm not ready to figure out how to change the params to make it work.	3 years ago
Micah Snyder	a45bf34802	GH Actions: Fix build on Windows ... A breaking change was made to the run-cmake action. This commit pins the cmake actions to a specific version so they work again.	4 years ago
Micah Snyder	f75c3b3b4c	Github actions: Update package list before install ... To fix issues with installing older packages that may have been replaced	4 years ago
Micah Snyder	9451224323	Test: ClamDScan ExcludePath; Valgrind ... Adds a basic test to validate that ExcludePath correctly excludes a subdirectory but does not exclude subsequent files. As with the other ClamD/Scan tests, it will test in each mode: regular, stream, and fdpass (if available). Unlike the other tests, this one tests ClamDScan with Valgrind instead of ClamD. Refactored the clamd_test.py file to reduce duplicate code, and support enabling and disabling valgrind when running ClamDScan and ClamD. Add pytest to the github actions environments because the results when using pytest are far easier to read.	4 years ago
Micah Snyder	d57d898652	Github actions: Fix build after repo name change	4 years ago
Micah Snyder	c81968d3a7	GitHub Actions testing on Ubuntu, Mac, & Windows ... Updates to fix issues in the CMake install instructions. Updates the README.md to indicate that CMake is now preferred Adds a GitHub Actions badge, Discord badge, and logo to the README.md. CMake: - Renamed ENABLE_DOCS to ENABLE_MAN_PAGES. - Fixed build issue when milter isn't enabled on Linux. Changed the default to build milter on non-macOS, non-Windows operating systems. - Fix LD_LIBRARY_PATH for tests including on macOS where LD_LIBRARY_PATH and DYLD_LIBRARY_PATH must be manually propagated to subprocesses. - Use UNKNOWN IMPORTED library instead of INTERFACE IMPORTED library for pdcurses, but still use INTERFACE IMPORTED for ncurses. UNKNOWN IMPORTED appears to be required so that we can use $<TARGET_FILE_DIR:Curses::curses> to collected the pdcurses library at install time on Windows. - When building with vcpkg on Windows, CMake will automatically install your app local dependencies (aka the DLL runtime dependencies). Meanwhile, file(GET_RUNTIME_DEPENDENCIES ...) doesn't appear to work correctly with vcpkg packages. The solution is to use a custom target that has CMake perform a local install to the unit_tests directory when using vcpkg. This is in fact far easier than using GET_RUNTIME_DEPENDENCIES in the unit_tests for assembling the test environment but we can't use this method for the non-vcpkg install because it won't collect checkDynamic.dll for us because we don't install our tests. We also can't link with the static check.lib because the static check.lib has pthreads symbols linked in and will conflict with our pthread.dll. TL;DR: We'll continue to use file(GET_RUNTIME_DEPENDENCIES ...) for assembling the test enviornment on non-vcpkg builds, and use the local install method for vcpkg builds. testcase.py: Wrapped a Pathlib.unlink() call in exception handling as the missing_ok optional parameter requires a Python version too new for common use. Remove localtime_r from win32 compat lib. localtime_r may be present in libcheck when building with vcpkg and while making it a static function would also solve the issue, using localtime_s instead like we do everywhere else should work just fine. check_clamd: Limited the max # of connections for the stress test on Mac to 850, to address issues found testing on macos-latest on GitHub Actions.	4 years ago