[cmake-developers] Review Request: Topic ExternalProject_exclude-from-all

Daniele E. Domenichelli daniele.domenichelli at gmail.com
Thu Mar 27 14:23:56 EDT 2014


On 27/03/14 18:50, Brad King wrote:
> On 03/27/2014 12:24 PM, Daniele E. Domenichelli wrote:
>> +  if(";${ARGN};" MATCHES ";EXCLUDE_FROM_ALL;([^;]*);")
>> +    set(exclude_from_all ${CMAKE_MATCH_1})
>> +  endif()
>> +  if(NOT exclude_from_all)
>> +    set(all ALL)
>> +  endif()

Sorry, I made 2 force-pushes (after each of your e-mails) and perhaps
you fetched the branch in the time between the first and the second
one...
The updated code after your second e-mail should be this:

+  get_property(exclude_from_all TARGET ${name} PROPERTY _EP_EXCLUDE_FROM_ALL)
+  if(exclude_from_all)
+    set_property(TARGET ${name} PROPERTY EXCLUDE_FROM_ALL TRUE)
+  endif()


Daniele



More information about the cmake-developers mailing list