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

Unified Diff: third_party/boringssl/boringssl.gyp

Issue 401153002: Switch to BoringSSL. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase across DEPS change Created 6 years, 5 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/boringssl/BUILD.gn ('k') | third_party/boringssl/boringssl_nacl.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/boringssl/boringssl.gyp
diff --git a/third_party/boringssl/boringssl.gyp b/third_party/boringssl/boringssl.gyp
index 19e3fcb69587710a741a35bdbfc09eb4791f55a7..fa7c5b6705fbc8e907f01d2cc9c453dfd0fb0a17 100644
--- a/third_party/boringssl/boringssl.gyp
+++ b/third_party/boringssl/boringssl.gyp
@@ -9,7 +9,7 @@
'targets': [
{
'target_name': 'boringssl',
- 'type': 'static_library',
+ 'type': '<(component)',
'includes': [
'boringssl.gypi',
],
@@ -52,6 +52,19 @@
['target_arch != "arm" and target_arch != "ia32" and target_arch != "x64"', {
'defines': [ 'OPENSSL_NO_ASM' ],
}],
+ ['component == "shared_library"', {
+ 'xcode_settings': {
+ 'GCC_SYMBOLS_PRIVATE_EXTERN': 'NO', # no -fvisibility=hidden
+ },
+ 'cflags!': [ '-fvisibility=hidden' ],
+ 'conditions': [
+ ['os_posix == 1 and OS != "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' ],
+ }],
+ ],
+ }],
],
'include_dirs': [
'src/include',
« no previous file with comments | « third_party/boringssl/BUILD.gn ('k') | third_party/boringssl/boringssl_nacl.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698