Re: [cmake-developers] Pssible documentation error in install(TARGETS ) signature

2016-02-17 Thread Brad King
On 02/16/2016 05:42 PM, Daniel Wirtz wrote:
> if the order of the arguments matters: update documentation?

Thanks, updated:

 Help: Clarify install(TARGETS) INCLUDES DESTINATION option
 https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=c978223c

-Brad

-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers


[cmake-developers] Pssible documentation error in install(TARGETS ) signature

2016-02-16 Thread Daniel Wirtz

The current CMake version has an error in the Documentation (or a Bug in the 
Code, depending on how this turns out).
In https://cmake.org/cmake/help/v3.5/command/install.html#installing-targets,
the optional argument order is

[...]
[INCLUDES DESTINATION [ ...]]
[PERMISSIONS permissions...]
[CONFIGURATIONS [Debug|Release|...]]
[COMPONENT ]
[...]

if i use that in the order as documented, e.g.

install(TARGETS mytarget
EXPORT myexport
RUNTIME DESTINATION bin
COMPONENT Runtime
ARCHIVE DESTINATION lib
INCLUDES DESTINATION include
COMPONENT Runtime
),
the resulting cmake_install.cmake file will contain the install type 
"Unspecified" for my .lib archives (using windows)
i have to change the order of the arguments (using COMPONENT before INCLUDES 
DESTINATION):

install(TARGETS mytarget
EXPORT myexport
RUNTIME DESTINATION bin
COMPONENT Runtime
ARCHIVE DESTINATION lib
COMPONENT Runtime # Note swap here
INCLUDES DESTINATION include # Note swap here
)

then it correctly puts "Runtime" in the cmake_install.cmake file.
if the order of the arguments matters: update documentation?
if the order does not matter: could be a bug in the parser (two-element keyword 
INCLUDES DESTINATION?)?

any opinions welcome ..
Best,
Daniel

--
Dr. Daniel Wirtz
Dipl. Math. Dipl. Inf.
SRC SimTech
Pfaffenwaldring 5a, D-70569 Stuttgart
+49 (711) 685-60044

--

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers