Tags: patch
Hi,
Since version 3.6.2, there is a proper check for the availability of
readdir_r. So we could leave things as they are and wait for libc to
remove readdir_r at which point the warnings will also vanish. Or we
could force dcmtk to no longer use readdir_r by disabling the check. I
attached a patch for the latter.
Best,
Fabian
From: Fabian Klötzl <kloe...@evolbio.mpg.de>
Date: Fri, 22 Dec 2017 12:55:35 +0000
Subject: do not use readdir_r anymore
---
CMake/GenerateDCMTKConfigure.cmake | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/CMake/GenerateDCMTKConfigure.cmake b/CMake/GenerateDCMTKConfigure.cmake
index dc1e920..924586a 100755
--- a/CMake/GenerateDCMTKConfigure.cmake
+++ b/CMake/GenerateDCMTKConfigure.cmake
@@ -616,7 +616,8 @@ ENDIF(WIN32 AND NOT CYGWIN)
CHECK_FUNCTIONWITHHEADER_EXISTS("_fpclassf(0.0f)" "${HEADERS}" HAVE_PROTOTYPE__FPCLASSF)
CHECK_FUNCTIONWITHHEADER_EXISTS("getgrnam_r((char*)0,(group*)0,(char*)0,0,(group**)0)" "${HEADERS}" HAVE_GETGRNAM_R)
CHECK_FUNCTIONWITHHEADER_EXISTS("getpwnam_r((char*)0,(passwd*)0,(char*)0,0,(passwd**)0)" "${HEADERS}" HAVE_GETPWNAM_R)
- CHECK_FUNCTIONWITHHEADER_EXISTS("readdir_r((DIR*)0,(dirent*)0,(dirent**)0)" "${HEADERS}" HAVE_READDIR_R)
+ #CHECK_FUNCTIONWITHHEADER_EXISTS("readdir_r((DIR*)0,(dirent*)0,(dirent**)0)" "${HEADERS}" HAVE_READDIR_R)
+ set(HAVE_READDIR_R 0)
CHECK_FUNCTIONWITHHEADER_EXISTS(nanosleep "${HEADERS}" HAVE_PROTOTYPE_NANOSLEEP)
CHECK_FUNCTIONWITHHEADER_EXISTS("&passwd::pw_gecos" "${HEADERS}" HAVE_PASSWD_GECOS)
