Am Freitag, 13. Mai 2016, 11:16:28 schrieb Brad King: > On 05/13/2016 11:02 AM, Rolf Eike Beer wrote: > > Should all be done now. > > Thanks. > > The test fails for me with > > CMake Error at /.../Modules/FindPkgConfig.cmake:239 > (set_target_properties): set_target_properties called with incorrect number > of arguments. > At least part of the problem is here: > > + if(${_prefix}_INCLUDE_DIRS) > > + list(APPEND _props INTERFACE_INCLUDE_DIRECTORIES > > "${${_prefix}_INCLUDE_DIRS}") + endif() > > + if(_libs) > > + list(APPEND _props INTERFACE_LINK_LIBRARIES "${_libs}") > > + endif() > > + if(${_prefix}_CFLAGS_OTHER) > > + list(APPEND _props INTERFACE_COMPILE_OPTIONS > > "${${_prefix}_CFLAGS_OTHER}") + endif() > > + set_target_properties(PkgConfig::${_prefix} PROPERTIES ${_props}) > > If any of the property values is empty the `${_props}` will > remove it from the argument list to set_target_properties. > Can you just use > > set_property(TARGET PkgConfig::${_prefix} PROPERTY ...) > > in each `if()` block?
I had this before, but I thought I avoid multiple target lookups. You should only ever see that message if none of the branches is taken I think. And that should never happen, or the if before is wrong. Can you add a message() and show what is actually in _props there? Greetings, Eike
signature.asc
Description: This is a digitally signed message part.
-- 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