View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0014082CMakeModulespublic2013-04-15 05:202016-06-10 14:31
ReporterJochen Wilhelmy 
Assigned ToKitware Robot 
PlatformOSWindowsOS Version
Product VersionCMake 
Target VersionFixed in Version 
Summary0014082: FindZLIB.cmake does not find debug version of zlib library
DescriptionOnly release library is found which produces linker warnings on windows
Steps To Reproducefind_package(zlib)
Additional InformationA fixed FindZLIB.cmake is attached that searches for the debug version separately and returns both in ZLIB_LIBRARIES, e.g. as debug;zlibd.lib;optimized;zlib.lib
TagsNo tags attached.
Attached Files? file icon FindZLIB.cmake [^] (4,082 bytes) 2013-04-15 05:20
? file icon FindZLIB_2.cmake [^] (3,693 bytes) 2013-04-15 09:20
? file icon SelectLibraryConfigurations.cmake [^] (4,256 bytes) 2013-04-15 09:20


Rolf Eike Beer (developer)
2013-04-15 07:12

Please use SelectLibraryConfigurations module.
Jochen Wilhelmy (reporter)
2013-04-15 09:23

Now I used SelectLibraryConfigurations.cmake and used FindBZip2.cmake as a template.

I also patched SelectLibraryConfigurations.cmake because it duplicated the libraries. I replaced

set( ${basename}_LIBRARY )

which restores a cached entry from a previous run of cmake by

set( ${basename}_LIBRARY "" )
Jochen Wilhelmy (reporter)
2013-04-15 09:24

which one is correct
Rolf Eike Beer (developer)
2013-04-16 02:37

The one you search for using find_library() is the singular one. The one to include is the plural one, because it may contain multiple things (different configurations like in this case, or just the library and it's dependencies).

You do not need to if-guard the find_library() calls. If the output variable of find_library() is already set it simply does nothing.

You must not remove the ZLIB_ROOT code, as we need to keep compatibility with older releases.

Can you describe the problem in SelectLibraryConfigurations a bit more? Under which circumstances does it duplicate entries?
Jochen Wilhelmy (reporter)
2013-04-16 04:36

so exposing ZLIB_INCLUDE_DIRS and ZLIB_LIBRARIES (both plural) in FindZLIB.cmake is correct? Some FindXXX.cmake scripts expose XXX_INCLUDE_DIR (e.g. FindBZip2.cmake).

The problem in SelectLibraryConfigurations appears in the second run of cmake where set( ${basename}_LIBRARY ) sets the variable to the value of the first run and not to empty.
so after the first run if ${basename} is ZLIB then ZLIB_LIBRARY is
after the second run ZLIB_LIBRARY is

Is it possible that you fix FindZLiB.cmake, FindPNG.cmake, FindJPEG.cmake and findTIFF.cmake to use SelectLibraryConfigurations because I may break the one or other compatibility requirement. But I can give it another try ;-)
Rolf Eike Beer (developer)
2013-05-11 09:16

Well, Modules/readme.txt has the way it should be, and we try hard that it becomes that way on any new modules or if modules need reworking. But a harder requirement is backwards compatibility, so if old modules have a different behaviour they will keep this (maybe getting the new one additionally).

I'll look into the SLC issue.
Stephen Kelly (developer)
2013-05-29 02:51

The SLC issue may have been fixed with commit a2099a8e8d2b97d2a2f91cbc06a4078f7c5b7139
Stephen Kelly (developer)
2013-11-02 11:36

Is there anything blocking this now?
Kitware Robot (administrator)
2016-06-10 14:28

Resolving issue as `moved`.

This issue tracker is no longer used. Further discussion of this issue may take place in the current CMake Issues page linked in the banner at the top of this page.

 Issue History
Date Modified Username Field Change
2013-04-15 05:20 Jochen Wilhelmy New Issue
2013-04-15 05:20 Jochen Wilhelmy File Added: FindZLIB.cmake
2013-04-15 07:12 Rolf Eike Beer Note Added: 0032828
2013-04-15 09:20 Jochen Wilhelmy File Added: FindZLIB_2.cmake
2013-04-15 09:20 Jochen Wilhelmy File Added: SelectLibraryConfigurations.cmake
2013-04-15 09:23 Jochen Wilhelmy Note Added: 0032831
2013-04-15 09:24 Jochen Wilhelmy Note Added: 0032832
2013-04-16 02:37 Rolf Eike Beer Note Added: 0032847
2013-04-16 04:36 Jochen Wilhelmy Note Added: 0032851
2013-05-11 09:16 Rolf Eike Beer Note Added: 0033028
2013-05-29 02:51 Stephen Kelly Note Added: 0033149
2013-11-02 11:36 Stephen Kelly Note Added: 0034334
2016-06-10 14:28 Kitware Robot Note Added: 0042267
2016-06-10 14:28 Kitware Robot Status new => resolved
2016-06-10 14:28 Kitware Robot Resolution open => moved
2016-06-10 14:28 Kitware Robot Assigned To => Kitware Robot
2016-06-10 14:31 Kitware Robot Status resolved => closed

Copyright © 2000 - 2018 MantisBT Team