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

Unified Diff: openssl.gyp

Issue 144383010: A direct dependent of this project should also use the x64 version of (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/deps/third_party/openssl/
Patch Set: Add '.' back to openssl_include_dirs Created 6 years, 10 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: openssl.gyp
===================================================================
--- openssl.gyp (revision 244037)
+++ openssl.gyp (working copy)
@@ -21,6 +21,9 @@
'openssl/crypto/modes',
'openssl/include',
],
+ 'openssl_public_include_dirs': [
+ 'openssl/include',
+ ],
},
'sources': [
'<@(openssl_common_sources)',
@@ -83,6 +86,7 @@
'variables': {
# Ensure the 64-bit opensslconf.h header is used.
'openssl_include_dirs+': [ 'config/x64' ],
+ 'openssl_public_include_dirs+': [ 'config/x64' ],
Ryan Sleevi 2014/02/05 02:13:03 Is the public API affected by this (eg: including
wtc 2014/02/05 17:14:32 opensslconf.h is the only header under the config/
},
}],
['component == "shared_library"', {
@@ -107,7 +111,7 @@
],
'direct_dependent_settings': {
'include_dirs': [
- 'openssl/include',
+ '<@(openssl_public_include_dirs)',
],
},
},
« 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