MantisBT - CMake
View Issue Details
0015323CMakeCMakepublic2014-12-22 15:452015-05-04 09:05
yaruopooner 
 
highminoralways
closedno change required 
MSVC2013 Win64Windows7 SP1
CMake 3.1 
 
0015323: Visual Studio 2013 %3B in IMPORTED_IMPLIB_<CONFIG>
Include "%3B" in the output of the "IMPORTED_IMPLIB_<CONFIG>".

This problem has occurred from cmake3.1.0.
3.0.2 don't have this issue.

I tried a few pattern.

set_property(TARGET additional_dependencies PROPERTY IMPORTED_IMPLIB_RELEASE foo.lib bar.lib)

or

set_property(TARGET additional_dependencies PROPERTY IMPORTED_IMPLIB_RELEASE "foo.lib;bar.lib")

but I got same output.

foo.lib%3Bbar.lib
-console
cmake -G "Visual Studio 12 2013 Win64" .

-CMakeLists.txt(Excerpt)
add_library(additional_dependencies SHARED IMPORTED)
set_property(TARGET additional_dependencies PROPERTY IMPORTED_IMPLIB_RELEASE foo.lib bar.lib)

-output
foo.lib%3Bbar.lib
No tags attached.
Issue History
2014-12-22 15:45yaruopoonerNew Issue
2014-12-22 16:14Brad KingNote Added: 0037511
2014-12-22 16:14Brad KingStatusnew => resolved
2014-12-22 16:14Brad KingResolutionopen => no change required
2014-12-22 16:16Brad KingNote Added: 0037512
2015-05-04 09:05Robert MaynardNote Added: 0038707
2015-05-04 09:05Robert MaynardStatusresolved => closed

Notes
(0037511)
Brad King   
2014-12-22 16:14   
The IMPORTED_IMPLIB is documented to be a single value. Anything else was supported only by accident and would be not likely to work in non-VS generators.

If you want to import multiple libraries then each should get its own imported target, and dependencies can go in INTERFACE_LINK_LIBRARIES.

Please ask on the mailing list:

 http://www.cmake.org/mailman/listinfo/cmake [^]

if you need further help.
(0037512)
Brad King   
2014-12-22 16:16   
Re 0015323:0037511: More specifically IMPORTED_IMPLIB is documented as a *full path* to a single import library file:

 http://www.cmake.org/cmake/help/v3.0/prop_tgt/IMPORTED_IMPLIB.html [^]
(0038707)
Robert Maynard   
2015-05-04 09:05   
Closing resolved issues that have not been updated in more than 4 months.