Add missing defines = [] to boringssl/BUILD.gn

third_party/boringssl/BUILD.gn

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # Config for us and everybody else depending on BoringSSL.
 config("openssl_config") {
   include_dirs = []
   include_dirs += [ "src/include" ]
 }
 
@@ skipping 21 matching lines @@
 
   # Also gets the include dirs from :openssl_config
   include_dirs = [
     "src/include",
     # This is for arm_arch.h, which is needed by some asm files. Since the
     # asm files are generated and kept in a different directory, they
     # cannot use relative paths to find this file.
     "src/crypto",
   ]
 
+  defines = []

[brettw, 2014/07/24 22:21:36]

I'd personally put this with the cflags at the top, but either way is OK.

[scottmg, 2014/07/24 22:22:43]

Done.

   if (cpu_arch == "x64") {
     if (is_mac) {
       sources += gypi_values.boringssl_mac_x86_64_sources
     } else if (is_linux || is_android) {
       sources += gypi_values.boringssl_linux_x86_64_sources
     } else if (is_win) {
       sources += gypi_values.boringssl_win_x86_64_sources
     } else {
       defines += [ "OPENSSL_NO_ASM" ]
     }
@@ skipping 15 matching lines @@
     if (!is_win) {
       configs -= [ "//build/config/gcc:symbol_visibility_hidden" ]
     }
     if (is_posix && !is_mac) {
       # Avoid link failures on Linux x86-64.
       # See http://rt.openssl.org/Ticket/Display.html?id=2466&user=guest&pass=guest
       ldflags += [ "-Wl,-Bsymbolic" ]
     }
   }
 }