On 07/26/2013 04:17 PM, Alexander Neundorf wrote: > This should make sure that using this variable in a tll() call should suffice > to give the target the required include dirs and libraries.
Sounds good! I propose the name "Foo_LIBRARY_TARGETS" instead because the intended use case is only for libraries. > KCoreAddons_TARGETS set by it lists the following non-existant targets: Foo s/existant/existent/ > * Should the variable be mentioned in the documentation for find_package() ? Yes, since it is this command that enforces the error messages. > * There is the theoretical risk that there are Config.cmake files out there > which set a variable with that name already, but without the include dir > property. Should this be based on a policy, so that the error is only > generated if cmake 2.8.12 or higher is required ? I think with the name "<pkg>_LIBRARY_TARGETS" the risk is small. Thanks, -Brad -- Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Please keep messages on-topic and check the CMake FAQ at: http://www.cmake.org/Wiki/CMake_FAQ Follow this link to subscribe/unsubscribe: http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers
