[PATCH] BUILD: Fix detection of pthread with strict CFLAGS
by Lukas Slebodnik
ehlo,
If the configure was called with stricter flags (-Werror=unused-variable)
then configure script did not detect tread safe initialisation.
As a result of this client code was not build with mutexes.
conftest.c: In function 'main':
conftest.c:39:17: error: unused variable 'm' [-Werror=unused-variable]
pthread_mutex_t m = PTHREAD_MUTEX_INITIALIZER;
^
cc1: all warnings being treated as errors
configure:15331: $? = 1
configure:15338: WARNING: Pthread library not found! Clients will not be thread safe...
Simple patch is attached.
LS
8 years, 5 months
[PATCH] intg: Fix all PEP8 issues
by Nikolai Kondrashov
Hi everyone,
The attached patch fixes all PEP8 issues in src/tests/intg directory. I.e. the
integration tests. It was extracted from the "intg: Add more LDAP tests"
thread.
Nick
8 years, 5 months
[PATCH] Don't skip tests not checked with Valgrind + extra
by Nikolai Kondrashov
Hi everyone,
I've found an embarrassing logic failure in my Valgrind execution design in
CI: tests which were excluded from Valgrind checks weren't running at all,
instead of just running without Valgrind.
The first attached patch fixes that. The second patch makes CI handle test
executables which start with a dash. Not that we're likely to ever have any,
but just in case.
Sorry for overlooking this.
Nick
8 years, 6 months