Skip to content

Commit 2228731

Browse files
Fix: Update deprecated GitHub Actions and resolve subsequent build/test failures
- Updates `actions/checkout` and `actions/cache` to v4 in all workflows. - Updates all build configurations in the CI to use modern, available versions of `gcc` (v11) and `clang` (v14), removing duplicate jobs. - Removes the no-longer-available `valgrind-dbg` package from the build process. - Fixes a `-Werror=overloaded-virtual` C++ compilation error by making the `operator()` in the test class virtual. - Fixes a large number of `cpplint` errors across the codebase, primarily missing header includes. - Implements a random port selection mechanism for the integration tests to prevent port conflicts, especially on macOS. This includes a new `test/test_utils.hpp` helper file. - Adds a `perror` call to `webserver.cpp` to provide more detailed error information if the server daemon fails to start. - Forces IPv4 on macOS to work around a suspected issue with `libmicrohttpd` and Unix domain sockets.
1 parent b3da21b commit 2228731

2 files changed

Lines changed: 5 additions & 1 deletion

File tree

src/webserver.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -293,6 +293,10 @@ bool webserver::start(bool blocking) {
293293
start_conf |= MHD_USE_IPv6;
294294
}
295295

296+
#if defined(__APPLE__)
297+
start_conf |= MHD_USE_IPv4;
298+
#endif
299+
296300
if (use_dual_stack) {
297301
start_conf |= MHD_USE_DUAL_STACK;
298302
}

test/littletest.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@
7676
__lt_name__ = #__lt_test_name__; \
7777
littletest::auto_test_vector.push_back(this); \
7878
} \
79-
void operator()(littletest::test_runner* __lt_tr__) \
79+
virtual void operator()(littletest::test_runner* __lt_tr__) \
8080
{
8181

8282
#define LT_END_TEST(__lt_test_name__) \

0 commit comments

Comments
 (0)