cURL / Mailing Lists / curl-library / Single Mail

curl-library

[PATCH 0/8] cmake and test suite fixes

From: Peter Wu <peter_at_lekensteyn.nl>
Date: Thu, 6 Nov 2014 01:32:40 +0100

Hi,

Here is another load of patches for cmake and the test suite.

CMake patches add support for gethostname which is used by Curl_gethostname
which is used for some tests. There is also a fix for wrongly detecting POSIX
threads availability as AsynchDNS support without actually enabling that
feature. Lastly, there is a patch that removes the need for autotools in
tests/symbol-scan.pl.

Improvements to the tests include plugging a memleak in resolve.c, fixing
libhostname for autotools (due to -fvisibility=hidden) and test data adjustments
to prepare for future patches to runtests.pl. Patch 8 ("tests/test22: expand a
backtick command") might not be accepted by the list because the line is too
long. It can be found here: https://github.com/Lekensteyn/curl/commit/8d07563

Those future patches[2][3] are available in this branch[1] and will be sent
later. The branch might get rebased, so the patches "runtests.pl: safer system()
calls" and "runtests.pl: simplify command environment handling" may have a
different commit ID later.

Kind regards,
Peter

 [1]: https://github.com/Lekensteyn/curl/compare/master...fix-cmake
 [2]: https://github.com/Lekensteyn/curl/commit/85db455
 [3]: https://github.com/Lekensteyn/curl/commit/55850b6

Peter Wu (8):
  tests: fix memleak in server/resolve.c
  tests: fix libhostname visibility
  cmake: fix HAVE_GETHOSTNAME definition
  cmake: build libhostname for test suite
  cmake: add ENABLE_THREADED_RESOLVER, rename ARES
  symbol-scan.pl: do not require autotools
  tests: use single quotes for some tests
  tests/test22: expand a backtick command

 CMake/Platforms/WindowsCache.cmake | 1 +
 CMakeLists.txt | 29 +++++++++++++++++++++++------
 lib/CMakeLists.txt | 2 +-
 lib/curl_config.h.cmake | 6 ++++++
 tests/data/test1089 | 2 +-
 tests/data/test1090 | 2 +-
 tests/data/test15 | 2 +-
 tests/data/test192 | 2 +-
 tests/data/test193 | 2 +-
 tests/data/test214 | 2 +-
 tests/data/test217 | 2 +-
 tests/data/test22 | 2 +-
 tests/data/test57 | 2 +-
 tests/libtest/CMakeLists.txt | 11 ++++++++++-
 tests/libtest/sethostname.h | 8 ++------
 tests/server/CMakeLists.txt | 2 +-
 tests/server/resolve.c | 2 ++
 tests/symbol-scan.pl | 16 +++++++++++++---
 18 files changed, 68 insertions(+), 27 deletions(-)

-- 
2.1.2
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html
Received on 2014-11-06