From 6a3379db62ccbce7fc9a859032955614e78e68bb Mon Sep 17 00:00:00 2001
From: Gil <mcg@google.com>
Date: Wed, 17 Jan 2018 10:24:27 -0800
Subject: Add a cc_library to the CMake build (#670)

* Rewrite cc_test to take named arguments

Cut down on build file verbosity by having cc_test take SOURCES and
DEPENDS. The separate invocation of target_link_libraries is no longer
necessary.

* Add a cc_library rule to parallel cc_test

This cuts down on build file verbosity.

* Automatically add OBJC_FLAGS to cc_libraries if applicable

* Exclude platform-specific libraries from 'all'

This is makes it possible to declare this kind of library
unconditionally. Usage within a test or as a dependency will actually
trigger building.

* Restore secure_random_test.cc; clean-up comments
---
 cmake/utils.cmake | 80 +++++++++++++++++++++++++++++++++++++++++++++++++------
 1 file changed, 72 insertions(+), 8 deletions(-)

(limited to 'cmake')

diff --git a/cmake/utils.cmake b/cmake/utils.cmake
index 54044d6..1c3cbd6 100644
--- a/cmake/utils.cmake
+++ b/cmake/utils.cmake
@@ -12,16 +12,80 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-# Defines a new test executable and does all the things we want done with
-# tests:
+include(CMakeParseArguments)
+
+# cc_library(
+#   target
+#   SOURCES sources...
+#   DEPENDS libraries...
+# )
+#
+# Defines a new library target with the given target name, sources, and dependencies.
+function(cc_library name)
+  set(flag EXCLUDE_FROM_ALL)
+  set(multi DEPENDS SOURCES)
+  cmake_parse_arguments(ccl "${flag}" "" "${multi}" ${ARGN})
+
+  add_library(
+    ${name}
+    ${ccl_SOURCES}
+  )
+  add_objc_flags(${name} ccl)
+  target_link_libraries(
+    ${name}
+    PUBLIC
+    ${ccl_DEPENDS}
+  )
+
+  if(ccl_EXCLUDE_FROM_ALL)
+    set_property(
+      TARGET ${name}
+      PROPERTY EXCLUDE_FROM_ALL ON
+    )
+  endif()
+
+endfunction()
+
+# cc_test(
+#   target
+#   SOURCES sources...
+#   DEPENDS libraries...
+# )
 #
-#   * add_executable (with the given arguments)
-#   * add_Test - defines a test with the same name
-#   * declares that the test links against gtest
-#   * adds the executable as a dependency of the `check` target.
+# Defines a new test executable target with the given target name, sources, and
+# dependencies.  Implicitly adds DEPENDS on GTest::GTest and GTest::Main.
 function(cc_test name)
-  add_executable(${name} ${ARGN})
+  set(multi DEPENDS SOURCES)
+  cmake_parse_arguments(cct "" "" "${multi}" ${ARGN})
+
+  list(APPEND cct_DEPENDS GTest::GTest GTest::Main)
+
+  add_executable(${name} ${cct_SOURCES})
+  add_objc_flags(${name} cct)
   add_test(${name} ${name})
 
-  target_link_libraries(${name} GTest::GTest GTest::Main)
+  target_link_libraries(${name} ${cct_DEPENDS})
+endfunction()
+
+# add_objc_flags(target sources...)
+#
+# Adds OBJC_FLAGS to the compile options of the given target if any of the
+# sources have filenames that indicate they are are Objective-C.
+function(add_objc_flags target)
+  set(_has_objc OFF)
+
+  foreach(source ${ARGN})
+    get_filename_component(ext ${source} EXT)
+    if((ext STREQUAL ".m") OR (ext STREQUAL ".mm"))
+      set(_has_objc ON)
+    endif()
+  endforeach()
+
+  if(_has_objc)
+    target_compile_options(
+      ${target}
+      PRIVATE
+      ${OBJC_FLAGS}
+    )
+  endif()
 endfunction()
-- 
cgit v1.2.3