6 Dec
2013
6 Dec
'13
2:54 a.m.
All,
I gave a pass at compiling R14 on the newly released Slackware
14.1, which contains cmake 2.8.12.
The arts package compiled with cmake warnings I never saw before
and tdelibs FTBFS with the same warnings. The arts warnings:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in flow/gsl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in flow/gsl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
tdelibs:
CMake Warning (dev) in tdecore/tdehw/CMakeLists.txt:
CMake Warning (dev) in tdecore/tdehw/networkbackends/network-
manager/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in tdecore/svgicons/CMakeLists.txt:
CMake Warning (dev) in tdeio/tdeio/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in tdehtml/java/CMakeLists.txt:
Following the colon of each warning:
Common to both arts and tdelibs:
CMake Warning (dev) in libltdl/CMakeLists.txt:
Policy CMP0022 is not set: INTERFACE_LINK_LIBRARIES defines the
link
interface. Run "cmake --help-policy CMP0022" for policy details.
Use the
cmake_policy command to set the policy and suppress this warning.
Static library target "ltdlc-static" has a
INTERFACE_LINK_LIBRARIES
property. This should be preferred as the source of the link
interface for
this library. Ignoring the property and using the link
implementation as
the link interface instead.
This warning is for project developers. Use -Wno-dev to suppress
it.
If I understand correctly from reading around the web, the cause is
cmake 2.8.12 changed LINK_INTERFACE_LIBRARIES to
INTERFACE_LINK_LIBRARIES.
If that is correct, then seems TDEMacros.cmake:696 needs to be
updated to handle both variables:
target_link_libraries( ${_target} LINK_INTERFACE_LIBRARIES
${_shared_libs} )
I'm wild guessing something similar to this (I'm not a cmake guru):
If (${CMAKE_MAJOR_VERSION}.${CMAKE_MINOR_VERSION}.${
CMAKE_PATCH_VERSION } GREATER 2.8.11)
target_link_libraries( ${_target} INTERFACE_LINK_LIBRARIES
${_shared_libs} )
else()
target_link_libraries( ${_target} LINK_INTERFACE_LIBRARIES
${_shared_libs} )
endif()
I presume others will experience these same warnings and build
failures when they update to cmake 2.8.12.
Regardless, this requires a change to the common cmake sources,
which need to be propogated through the source tree.
Darrell
8 Dec
8 Dec
4:42 a.m.
New subject: [trinity-devel] tdelibs FTBFS: cmake 2.8.12: INTERFACE_LINK_LIBRARIES vs. LINK_INTERFACE_LIBRARIES
2013/12/6 Darrell Anderson <darrella(a)hushmail.com>
All,
I gave a pass at compiling R14 on the newly released Slackware
14.1, which contains cmake 2.8.12.
The arts package compiled with cmake warnings I never saw before
and tdelibs FTBFS with the same warnings. The arts warnings:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in flow/gsl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in flow/gsl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
tdelibs:
CMake Warning (dev) in tdecore/tdehw/CMakeLists.txt:
CMake Warning (dev) in tdecore/tdehw/networkbackends/network-
manager/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in tdecore/svgicons/CMakeLists.txt:
CMake Warning (dev) in tdeio/tdeio/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in tdehtml/java/CMakeLists.txt:
Following the colon of each warning:
Common to both arts and tdelibs:
CMake Warning (dev) in libltdl/CMakeLists.txt:
Policy CMP0022 is not set: INTERFACE_LINK_LIBRARIES defines the
link
interface. Run "cmake --help-policy CMP0022" for policy details.
Use the
cmake_policy command to set the policy and suppress this warning.
Static library target "ltdlc-static" has a
INTERFACE_LINK_LIBRARIES
property. This should be preferred as the source of the link
interface for
this library. Ignoring the property and using the link
implementation as
the link interface instead.
This warning is for project developers. Use -Wno-dev to suppress
it.
If I understand correctly from reading around the web, the cause is
cmake 2.8.12 changed LINK_INTERFACE_LIBRARIES to
INTERFACE_LINK_LIBRARIES.
If that is correct, then seems TDEMacros.cmake:696 needs to be
updated to handle both variables:
target_link_libraries( ${_target} LINK_INTERFACE_LIBRARIES
${_shared_libs} )
I'm wild guessing something similar to this (I'm not a cmake guru):
If (${CMAKE_MAJOR_VERSION}.${CMAKE_MINOR_VERSION}.${
CMAKE_PATCH_VERSION } GREATER 2.8.11)
target_link_libraries( ${_target} INTERFACE_LINK_LIBRARIES
${_shared_libs} )
else()
target_link_libraries( ${_target} LINK_INTERFACE_LIBRARIES
${_shared_libs} )
endif()
I presume others will experience these same warnings and build
failures when they update to cmake 2.8.12.
Regardless, this requires a change to the common cmake sources,
which need to be propogated through the source tree.
Darrell
I can confirm the issue. There are warnings but everything builds fine for
me.
IMO, the better fix would be something like next snip (not tested yet):
cmake_policy(PUSH)
cmake_policy(SET CMP0022 OLD)
target_link_libraries( ${_target} LINK_INTERFACE_LIBRARIES ${_shared_libs} )
cmake_policy(POP)
4:48 a.m.
New subject: [trinity-devel] tdelibs FTBFS: cmake 2.8.12: INTERFACE_LINK_LIBRARIES vs. LINK_INTERFACE_LIBRARIES
2013/12/6 Darrell Anderson
<darrella(a)hushmail.com>
Just as a word of warning, this would push R14 to January 2014 at the
earliest due to the time required to rebuild the entire package set for
Debian/Ubuntu.
Tim
All,
I gave a pass at compiling R14 on the newly released Slackware
14.1, which contains cmake 2.8.12.
The arts package compiled with cmake warnings I never saw before
and tdelibs FTBFS with the same warnings. The arts warnings:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in flow/gsl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in flow/gsl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
tdelibs:
CMake Warning (dev) in tdecore/tdehw/CMakeLists.txt:
CMake Warning (dev) in tdecore/tdehw/networkbackends/network-
manager/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in tdecore/svgicons/CMakeLists.txt:
CMake Warning (dev) in tdeio/tdeio/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in tdehtml/java/CMakeLists.txt:
Following the colon of each warning:
Common to both arts and tdelibs:
CMake Warning (dev) in libltdl/CMakeLists.txt:
Policy CMP0022 is not set: INTERFACE_LINK_LIBRARIES defines the
link
interface. Run "cmake --help-policy CMP0022" for policy details.
Use the
cmake_policy command to set the policy and suppress this warning.
Static library target "ltdlc-static" has a
INTERFACE_LINK_LIBRARIES
property. This should be preferred as the source of the link
interface for
this library. Ignoring the property and using the link
implementation as
the link interface instead.
This warning is for project developers. Use -Wno-dev to suppress
it.
If I understand correctly from reading around the web, the cause is
cmake 2.8.12 changed LINK_INTERFACE_LIBRARIES to
INTERFACE_LINK_LIBRARIES.
If that is correct, then seems TDEMacros.cmake:696 needs to be
updated to handle both variables:
target_link_libraries( ${_target} LINK_INTERFACE_LIBRARIES
${_shared_libs} )
I'm wild guessing something similar to this (I'm not a cmake guru):
If (${CMAKE_MAJOR_VERSION}.${CMAKE_MINOR_VERSION}.${
CMAKE_PATCH_VERSION } GREATER 2.8.11)
target_link_libraries( ${_target} INTERFACE_LINK_LIBRARIES
${_shared_libs} )
else()
target_link_libraries( ${_target} LINK_INTERFACE_LIBRARIES
${_shared_libs} )
endif()
I presume others will experience these same warnings and build
failures when they update to cmake 2.8.12.
Regardless, this requires a change to the common cmake sources,
which need to be propogated through the source tree.
Darrell
I can confirm the issue. There are warnings but everything builds fine for
me.
IMO, the better fix would be something like next snip (not tested yet):
cmake_policy(PUSH)
cmake_policy(SET CMP0022 OLD)
target_link_libraries( ${_target} LINK_INTERFACE_LIBRARIES ${_shared_libs}
)
cmake_policy(POP)
18 Dec
18 Dec
9:05 p.m.
New subject: [trinity-devel] tdelibs FTBFS: cmake 2.8.12: INTERFACE_LINK_LIBRARIES vs. LINK_INTERFACE_LIBRARIES
On Friday 06 of December 2013 03:54:49 Darrell Anderson wrote:
All,
I gave a pass at compiling R14 on the newly released Slackware
14.1, which contains cmake 2.8.12.
The arts package compiled with cmake warnings I never saw before
and tdelibs FTBFS with the same warnings. The arts warnings:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in flow/gsl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in flow/gsl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
tdelibs:
CMake Warning (dev) in tdecore/tdehw/CMakeLists.txt:
CMake Warning (dev) in tdecore/tdehw/networkbackends/network-
manager/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in tdecore/svgicons/CMakeLists.txt:
CMake Warning (dev) in tdeio/tdeio/CMakeLists.txt:
CMake Warning (dev) in libltdl/CMakeLists.txt:
CMake Warning (dev) in tdehtml/java/CMakeLists.txt:
Following the colon of each warning:
Common to both arts and tdelibs:
CMake Warning (dev) in libltdl/CMakeLists.txt:
Policy CMP0022 is not set: INTERFACE_LINK_LIBRARIES defines the
link
interface. Run "cmake --help-policy CMP0022" for policy details.
Use the
cmake_policy command to set the policy and suppress this warning.
Static library target "ltdlc-static" has a
INTERFACE_LINK_LIBRARIES
property. This should be preferred as the source of the link
interface for
this library. Ignoring the property and using the link
implementation as
the link interface instead.
This warning is for project developers. Use -Wno-dev to suppress
it.
If I understand correctly from reading around the web, the cause is
cmake 2.8.12 changed LINK_INTERFACE_LIBRARIES to
INTERFACE_LINK_LIBRARIES.
If that is correct, then seems TDEMacros.cmake:696 needs to be
updated to handle both variables:
target_link_libraries( ${_target} LINK_INTERFACE_LIBRARIES
${_shared_libs} )
I'm wild guessing something similar to this (I'm not a cmake guru):
If (${CMAKE_MAJOR_VERSION}.${CMAKE_MINOR_VERSION}.${
CMAKE_PATCH_VERSION } GREATER 2.8.11)
target_link_libraries( ${_target} INTERFACE_LINK_LIBRARIES
${_shared_libs} )
else()
target_link_libraries( ${_target} LINK_INTERFACE_LIBRARIES
${_shared_libs} )
endif()
I presume others will experience these same warnings and build
failures when they update to cmake 2.8.12.
Regardless, this requires a change to the common cmake sources,
which need to be propogated through the source tree.
Darrell
I dove into research the problem and I noticed that on Jessie is reported
another problem. For example:
CMake Warning (dev) in tdefx/CMakeLists.txt:
Policy CMP0022 is not set: INTERFACE_LINK_LIBRARIES defines the link
interface. Run "cmake --help-policy CMP0022" for policy details. Use the
cmake_policy command to set the policy and suppress this warning.
Target "tdefx-shared" has an INTERFACE_LINK_LIBRARIES property which
differs from its LINK_INTERFACE_LIBRARIES properties.
INTERFACE_LINK_LIBRARIES:
tqt;tqt-mt;Xrender;X11;c;tqt;tqt-mt;Xrender;X11;c
LINK_INTERFACE_LIBRARIES:
tqt;tqt-mt;Xrender;X11;c
This warning is for project developers. Use -Wno-dev to suppress it.
Therefore, I have prepared a patch that could solve both of these problems.
Please, test it.
Slavek
--
4033
days inactive
4045
days old
3 comments
4 participants
participants (4)
-
Darrell Anderson -
Fat-Zer -
Slávek Banko -
Timothy Pearson