Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(70)

Side by Side Diff: third_party/boringssl/BUILD.gn

Issue 412413002: Add missing defines = [] to boringssl/BUILD.gn (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: to top Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 # Config for us and everybody else depending on BoringSSL. 5 # Config for us and everybody else depending on BoringSSL.
6 config("openssl_config") { 6 config("openssl_config") {
7 include_dirs = [] 7 include_dirs = []
8 include_dirs += [ "src/include" ] 8 include_dirs += [ "src/include" ]
9 } 9 }
10 10
11 # Config internal to this build file. 11 # Config internal to this build file.
12 config("openssl_internal_config") { 12 config("openssl_internal_config") {
13 visibility = ":*" # Only targets in this file can depend on this. 13 visibility = ":*" # Only targets in this file can depend on this.
14 } 14 }
15 15
16 # The list of BoringSSL files is kept in boringssl.gypi. 16 # The list of BoringSSL files is kept in boringssl.gypi.
17 gypi_values = exec_script( 17 gypi_values = exec_script(
18 "//build/gypi_to_gn.py", 18 "//build/gypi_to_gn.py",
19 [ rebase_path("//third_party/boringssl/boringssl.gypi") ], 19 [ rebase_path("//third_party/boringssl/boringssl.gypi") ],
20 "scope", 20 "scope",
21 [ "//third_party/boringssl/boringssl.gypi" ]) 21 [ "//third_party/boringssl/boringssl.gypi" ])
22 22
23 component("boringssl") { 23 component("boringssl") {
24 sources = gypi_values.boringssl_lib_sources 24 sources = gypi_values.boringssl_lib_sources
25 25
26 direct_dependent_configs = [ ":openssl_config" ] 26 direct_dependent_configs = [ ":openssl_config" ]
27 27
28 cflags = [] 28 cflags = []
29 defines = []
29 30
30 configs -= [ "//build/config/compiler:chromium_code" ] 31 configs -= [ "//build/config/compiler:chromium_code" ]
31 configs += [ "//build/config/compiler:no_chromium_code" ] 32 configs += [ "//build/config/compiler:no_chromium_code" ]
32 33
33 # Also gets the include dirs from :openssl_config 34 # Also gets the include dirs from :openssl_config
34 include_dirs = [ 35 include_dirs = [
35 "src/include", 36 "src/include",
36 # This is for arm_arch.h, which is needed by some asm files. Since the 37 # This is for arm_arch.h, which is needed by some asm files. Since the
37 # asm files are generated and kept in a different directory, they 38 # asm files are generated and kept in a different directory, they
38 # cannot use relative paths to find this file. 39 # cannot use relative paths to find this file.
(...skipping 28 matching lines...) Expand all
67 if (!is_win) { 68 if (!is_win) {
68 configs -= [ "//build/config/gcc:symbol_visibility_hidden" ] 69 configs -= [ "//build/config/gcc:symbol_visibility_hidden" ]
69 } 70 }
70 if (is_posix && !is_mac) { 71 if (is_posix && !is_mac) {
71 # Avoid link failures on Linux x86-64. 72 # Avoid link failures on Linux x86-64.
72 # See http://rt.openssl.org/Ticket/Display.html?id=2466&user=guest&pass=gu est 73 # See http://rt.openssl.org/Ticket/Display.html?id=2466&user=guest&pass=gu est
73 ldflags += [ "-Wl,-Bsymbolic" ] 74 ldflags += [ "-Wl,-Bsymbolic" ]
74 } 75 }
75 } 76 }
76 } 77 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698