From 56353aaf42542efbc50c61d8fe779ceb7d627713 Mon Sep 17 00:00:00 2001 From: Evan Hunt Date: Sat, 10 Mar 2018 10:33:45 -0800 Subject: [PATCH] Fix compiler warnings and test failures when building without threads --- lib/isc/tests/task_test.c | 10 +++++++++- lib/isc/tests/timer_test.c | 17 +++++++++++++++++ 2 files changed, 26 insertions(+), 1 deletion(-) diff --git a/lib/isc/tests/task_test.c b/lib/isc/tests/task_test.c index 7da64f2ebe..2862603e75 100644 --- a/lib/isc/tests/task_test.c +++ b/lib/isc/tests/task_test.c @@ -37,11 +37,14 @@ */ static isc_mutex_t lock; -static isc_condition_t cv; int counter = 0; static int active[10]; static isc_boolean_t done = ISC_FALSE; +#ifdef ISC_PLATFORM_USETHREADS +static isc_condition_t cv; +#endif + static void set(isc_task_t *task, isc_event_t *event) { int *value = (int *) event->ev_arg; @@ -643,6 +646,10 @@ ATF_TC_BODY(task_exclusive, tc) { isc_test_end(); } +/* + * The remainder of these tests require threads + */ +#ifdef ISC_PLATFORM_USETHREADS /* * Max tasks test: * The task system can create and execute many tasks. Tests with 10000. @@ -1440,6 +1447,7 @@ ATF_TC_HEAD(purgeevent_notpurge, tc) { ATF_TC_BODY(purgeevent_notpurge, tc) { try_purgeevent(ISC_FALSE); } +#endif /* * Main diff --git a/lib/isc/tests/timer_test.c b/lib/isc/tests/timer_test.c index e360419b58..bf38442c44 100644 --- a/lib/isc/tests/timer_test.c +++ b/lib/isc/tests/timer_test.c @@ -29,6 +29,11 @@ #include "isctest.h" +/* + * This entire test requires threads. + */ +#ifdef ISC_PLATFORM_USETHREADS + /* * Helper functions */ @@ -559,6 +564,16 @@ ATF_TC_BODY(purge, tc) { isc_test_end(); } +#else +ATF_TC(untested); +ATF_TC_HEAD(untested, tc) { + atf_tc_set_md_var(tc, "descr", "skipping nsec3 test"); +} +ATF_TC_BODY(untested, tc) { + UNUSED(tc); + atf_tc_skip("DNSSEC not available"); +} +#endif /* * Main @@ -570,6 +585,8 @@ ATF_TP_ADD_TCS(tp) { ATF_TP_ADD_TC(tp, once_idle); ATF_TP_ADD_TC(tp, reset); ATF_TP_ADD_TC(tp, purge); +#else + ATF_TP_ADD_TC(tp, untested); #endif return (atf_no_error());