Revision: 70051
          http://sourceforge.net/p/brlcad/code/70051
Author:   starseeker
Date:     2017-08-05 12:59:46 +0000 (Sat, 05 Aug 2017)
Log Message:
-----------
More notes about cleaning up the need to override commands.  There are some 
like message that we've not considered yet, but long term we'd like to not have 
to do the overrides - it's an undocumented feature in CMake they'd prefer 
projects not use...

Modified Paths:
--------------
    brlcad/trunk/CMakeLists.txt

Modified: brlcad/trunk/CMakeLists.txt
===================================================================
--- brlcad/trunk/CMakeLists.txt 2017-08-05 12:47:14 UTC (rev 70050)
+++ brlcad/trunk/CMakeLists.txt 2017-08-05 12:59:46 UTC (rev 70051)
@@ -460,6 +460,16 @@
   # Override and wrap add_subdirectory.
   function(add_subdirectory name)
     _add_subdirectory(${name} ${ARGN})
+
+    # TODO - try to come up with some good way to do this bookkeeping without
+    # having to override add_subdirectory...
+    #
+    # see https://cmake.org/pipermail/cmake-developers/2015-July/025730.html
+    # for some work related to this...  the SOURCE_DIR and SOURCES properties
+    # went in in CMake 3.4.  Once we can require that as a minimum version, we
+    # can probably revisit at least part of this by getting a list of source
+    # directories for all targets at the end of the build and setting
+    # properties then.
     set_property(GLOBAL APPEND PROPERTY CMAKE_IGNORE_FILES 
"${CMAKE_CURRENT_SOURCE_DIR}/${name}")
     set_property(GLOBAL APPEND PROPERTY CMAKE_IGNORE_FILES 
"${CMAKE_CURRENT_SOURCE_DIR}/${name}/CMakeLists.txt")
     DISTCLEAN("${CMAKE_CURRENT_BINARY_DIR}/${name}/CMakeFiles")

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
BRL-CAD Source Commits mailing list
brlcad-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/brlcad-commits

Reply via email to