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

Side by Side Diff: openssl.gyp

Issue 13979005: Only make OpenSSL a component for Android component builds. (Closed) Base URL: https://chromium.googlesource.com/chromium/deps/openssl.git@master
Patch Set: component on android Created 7 years, 8 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 (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 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 { 5 {
6 'targets': [ 6 'targets': [
7 { 7 {
8 'target_name': 'openssl', 8 'target_name': 'openssl',
9 'type': '<(component)', 9 'type': 'static_library',
10 'includes': [ 10 'includes': [
11 # Include the auto-generated gypi file. 11 # Include the auto-generated gypi file.
12 '../../third_party/openssl/openssl.gypi' 12 '../../third_party/openssl/openssl.gypi'
13 ], 13 ],
14 'variables': { 14 'variables': {
15 'openssl_include_dirs': [ 15 'openssl_include_dirs': [
16 '.', 16 '.',
17 'openssl', 17 'openssl',
18 'openssl/crypto', 18 'openssl/crypto',
19 'openssl/crypto/asn1', 19 'openssl/crypto/asn1',
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
59 ['target_arch == "x64"', { 59 ['target_arch == "x64"', {
60 'sources': [ '<@(openssl_x86_64_sources)' ], 60 'sources': [ '<@(openssl_x86_64_sources)' ],
61 'sources!': [ '<@(openssl_x86_64_source_excludes)' ], 61 'sources!': [ '<@(openssl_x86_64_source_excludes)' ],
62 'defines': [ '<@(openssl_x86_64_defines)' ], 62 'defines': [ '<@(openssl_x86_64_defines)' ],
63 'defines!': [ 'OPENSSL_NO_ASM' ], 63 'defines!': [ 'OPENSSL_NO_ASM' ],
64 'variables': { 64 'variables': {
65 # Ensure the 64-bit opensslconf.h header is used. 65 # Ensure the 64-bit opensslconf.h header is used.
66 'openssl_include_dirs+': [ 'config/x64' ], 66 'openssl_include_dirs+': [ 'config/x64' ],
67 }, 67 },
68 }], 68 }],
69 ['component == "shared_library"', { 69 ['component == "shared_library" and OS=="android"', {
70 'conditions': [ 70 # On Android, build OpenSSL as a component. This is necessary as both
71 ['os_posix == 1 and OS != "mac" and OS != "ios"', { 71 # crypto and net call into OpenSSL but it initializes some static
72 'cflags!': ['-fvisibility=hidden'], 72 # state only once. However, this fails to compile on linux.
73 }], 73 'type': 'shared_library',
74 ], 74 'cflags!': ['-fvisibility=hidden'],
75 }], 75 }],
76 ['clang==1', { 76 ['clang==1', {
77 'cflags': [ 77 'cflags': [
78 # OpenSSL has a few |if ((foo == NULL))| checks. 78 # OpenSSL has a few |if ((foo == NULL))| checks.
79 '-Wno-parentheses-equality', 79 '-Wno-parentheses-equality',
80 # OpenSSL uses several function-style macros and then ignores the 80 # OpenSSL uses several function-style macros and then ignores the
81 # returned value. 81 # returned value.
82 '-Wno-unused-value', 82 '-Wno-unused-value',
83 ], 83 ],
84 }, { # Not clang. Disable all warnings. 84 }, { # Not clang. Disable all warnings.
(...skipping 12 matching lines...) Expand all
97 }, 97 },
98 }, 98 },
99 ], 99 ],
100 } 100 }
101 101
102 # Local Variables: 102 # Local Variables:
103 # tab-width:2 103 # tab-width:2
104 # indent-tabs-mode:nil 104 # indent-tabs-mode:nil
105 # End: 105 # End:
106 # vim: set expandtab tabstop=2 shiftwidth=2: 106 # vim: set expandtab tabstop=2 shiftwidth=2:
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