commit e7b2b13fd48e65797547ae469f4198eb5f2d96f5
author David Benjamin <davidben@google.com> Fri Jul 06 14:46:30 2018 -0400
committer Adam Langley <agl@google.com> Fri Jul 06 18:54:12 2018 +0000
tree 38a9bc4ac2652a732bd9f098b8daee030e30ba08
parent c7db3232c397aa3feb1d474d63a1c4dd674b6349

Add link to CMake bugfix.

Change-Id: I6d82cbb33a5ac2acf3f4e0993210e0bddba8f1ad
Reviewed-on: https://boringssl-review.googlesource.com/29644
Reviewed-by: Adam Langley <agl@google.com>

CMakeLists.txt

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 3ec6ac9..f58a7bb 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -174,9 +174,8 @@
   add_definitions(-D_CRT_SECURE_NO_WARNINGS)
   # VS 2017 and higher supports STL-only warning suppressions. Manually add to
   # C++ only to work around a CMake quoting bug when using NASM with the Visual
-  # Studio generator.
-  #
-  # TODO(davidben): Send a patch to CMake and link to it here.
+  # Studio generator. This will be fixed in CMake 3.13.0. See
+  # https://gitlab.kitware.com/cmake/cmake/merge_requests/2179
   add_compile_options($<$<COMPILE_LANGUAGE:CXX>:-D_STL_EXTRA_DISABLED_WARNINGS=4774\ 4987>)
 endif()