[CMake] Some minor issues for 2.6.0-RC-8
Bill Hoffman
bill.hoffman at kitware.com
Fri Apr 18 10:35:52 EDT 2008
Alan W. Irwin wrote:
> (1) In FindwxWidgets.cmake, wxWidgets_LIBRARIES is documented as
> "Path to the wxWidgets libraries". From the above result (before my macro
> transformed it) this documentation should be changed to something like
> "List of linker options to use when linking to the wxWidgets libraries".
Can the FindwxWidgets.cmake maintainer take a look at this? I think the
issue is that pkg-config is used. So, on windows where it is not used
then that statement is correct.
>
> (2) In CMP0003 OLD mode, the -pthread linker option triggers the warning.
> That is obviously incorrect, and only a list of actual libraries (specified
> only by -l options?) should trigger the warning.
I will look at the -pthread case, since it can not depend on a -L.
>
> (3) There is a "sequential" problem with these warnings so only the first
> one encountered is mentioned. Please go back to the old (CVS) method where
> (IIRC) a complete list of CMP0003 warnings was generated. It's a
> completely
> self-explanatory warning message so users should not complain if they
> encounter more than one of these messages.
>
It was too verbose, a project like paraview produced hundreds of
warnings, and in 99% of the cases, making CMP0003 to NEW is the fix that
needs to be done. The volume of output was confusing. And still the
message is confusing to people.
The only reason this warning is around is to let people know we may have
broken the link because -L is no longer used by default. It is not
meant to debug non full path linking which is not a problem and is still
supported as long as the directory is specified with link_directories.
-Bill
More information about the CMake
mailing list