On 05/13/2016 12:21 PM, Rolf Eike Beer wrote:
>> 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?
The problem is that _libs has more than one value, so set_target_properties
ends up with
PROPERTIES PROP1 v1 PROP2 v2a v2b PROP3 v3
^^^^^^^
Please use set_property(TARGET). It is much more robust, and this is not
exactly a performance-critical loop.
Thanks,
-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
