Opened 12 months ago
Closed 12 months ago
New point version.
Sun July 19, 2020: Released Check 0.15.1
based on hash db3ef470271e6e011f2cd1f5231a50375568bb46
* Fix warning in ptr macros with pointer to integer cast
* Fix various warnings in Check's unit tests
* Replace gnu_printf with printf in format __attribute__
* Fix warnings from Check's macros: "warning: too many arguments for format"
* Fix format specifiers that do not match the argument types
Looks like the libpipeline testsuite completely fails to compile with this version of check. The error is:
In file included from common.h:3,
argstr.c: In function 'test_argstr_trivial_fn':
argstr.c:40:2: error: too few arguments to function '_ck_assert_failed'
40 | fail_unless (!strcmp (cmd->name, "/bin/simple"));
/usr/include/check.h:502:27: note: declared here
502 | CK_DLL_EXP void CK_EXPORT _ck_assert_failed(const char *file, int line,
Note that there are lots of similar errors, all involving fail_unless and _ck_assert_failed, the only thing changing being the .c file name.
This complicated sed (maybe it can be simplified), allows the tests to build and pass (apply the sed in libpipeline):
sed -i '/\(fail_unless\|fail_if\) (\([^,]*([^)]*)[^,]*\|\[^,]*\));/s/);/,0);/' tests/*.c
FWIIW, there is a bug report for the pipeline tests issue now: https://savannah.nongnu.org/bugs/?58883.
I've already committed a patch:
It will be in my next commit, but I'm still waiting for glibc-2.32 which is overdue.
If upstream releases a new version, we can drop it.
Upstream has published a fix:
I've taken this for a second version of the patch:
Fixed at revision 12007.
Powered by Trac 1.5.3.dev0
By Edgewall Software
© 1998-2021 Gerard Beekmans.