[cmake-developers] [review] add_jar (UseJava) uses cmake_parse_arguments

Brad King brad.king at kitware.com
Tue Mar 26 13:44:24 EDT 2013


On 03/26/2013 01:35 PM, Matthew Woehlke wrote:
>> This should probably be
>> include(${CMAKE_CURRENT_LIST_DIR}/CMakeParseArguments.cmake)
> 
> ...or 'include("${CMAKE_ROOT}/Modules/CMakeParseArguments.cmake")'?
> 
> *Most* of the find modules appear to use CMAKE_CURRENT_LIST_DIR, but 
> there are some (not necessarily find-package) modules that use 
> CMAKE_ROOT, and some (mostly non-find) that rely on CMAKE_MODULE_PATH.

We switched to CMAKE_CURRENT_LIST_DIR some time ago.

Fixed:

 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=78b46403

I'll squash that in to the final version before merging to master and
then rebase the rest of your topic on it.

-Brad



More information about the cmake-developers mailing list