]> git.ipfire.org Git - thirdparty/curl.git/commit
tests/server: drop memdebug option
authorViktor Szakats <commit@vsz.me>
Sun, 15 Jun 2025 21:28:08 +0000 (23:28 +0200)
committerViktor Szakats <commit@vsz.me>
Sun, 15 Jun 2025 23:27:46 +0000 (01:27 +0200)
commitfffec3d7e90b032a03eee2192e68f8baf913b3b4
tree8cfbb71579cfcf09fb6a1b79c4d8f3ffd81bb1c4
parentee066732963b7051a8d2fd56fa91a4ce0b444bd5
tests/server: drop memdebug option

I added it just in case when removing enabled-by-default memdebug
from test servers. Apparently it broke after recent changes. It's
probably not a widely used feature and does not seem to be worth fixing.
It creates odd dependencies as the error message indicates:

```
[28/54] Building C object tests/server/CMakeFiles/servers.dir/__/__/lib/memdebug.c.obj
FAILED: tests/server/CMakeFiles/servers.dir/__/__/lib/memdebug.c.obj
[...]
lib/memdebug.c: In function 'curl_dbg_log':
lib/memdebug.c:465:12: error: implicit declaration of function 'mvsnprintf'; did you mean 'vsnprintf'? [-Wimplicit-function-declaration]
  465 |   nchars = mvsnprintf(buf, sizeof(buf), format, ap);
      |            ^~~~~~~~~~
      |            vsnprintf
lib/memdebug.c:465:12: warning: nested extern declaration of 'mvsnprintf' [-Wnested-externs]
```

This patch is dropping these build options:
- cmake: `ENABLE_SERVER_DEBUG`
- autotools: `--enable-server-debug` / `--disable-server-debug`

Follow-up to a16485a42ea5dabe6c327179a1678ad04d1c6b2f #16705

Closes #17629
CMakeLists.txt
configure.ac
docs/INSTALL-CMAKE.md
tests/server/CMakeLists.txt
tests/server/Makefile.am
tests/server/Makefile.inc