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

Unified Diff: crypto/BUILD.gn

Issue 881213004: Support building BoringSSL with NSS certificates. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/webcrypto/test/test_helpers.cc ('k') | crypto/crypto.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: crypto/BUILD.gn
diff --git a/crypto/BUILD.gn b/crypto/BUILD.gn
index d3456d344b99ff3abaf4002311bc639462dfcf33..27e786ce85aef51f83c4e0f71feeaf6150f4a130 100644
--- a/crypto/BUILD.gn
+++ b/crypto/BUILD.gn
@@ -136,9 +136,6 @@ component("crypto") {
"ec_signature_creator_nss.cc",
"encryptor_nss.cc",
"hmac_nss.cc",
- "nss_util.cc",
- "nss_util.h",
- "nss_util_internal.h",
"rsa_private_key_nss.cc",
"secure_hash_default.cc",
"signature_creator_nss.cc",
@@ -170,6 +167,16 @@ component("crypto") {
]
}
+ # Remove nss_util when NSS is used for neither the internal crypto library
+ # nor the platform certificate library.
+ if (use_openssl && !use_nss_certs) {
+ sources -= [
+ "nss_util.cc",
+ "nss_util.h",
+ "nss_util_internal.h",
+ ]
+ }
+
defines = [ "CRYPTO_IMPLEMENTATION" ]
}
@@ -229,12 +236,14 @@ test("crypto_unittests") {
"symmetric_key_unittest.cc",
]
- if (use_openssl || !is_linux) {
- sources -= [ "rsa_private_key_nss_unittest.cc" ]
+ # Remove nss_util when NSS is used for neither the internal crypto library
+ # nor the platform certificate library.
+ if (use_openssl && !use_nss_certs) {
+ sources -= [ "nss_util_unittest.cc" ]
}
if (use_openssl) {
- sources -= [ "nss_util_unittest.cc" ]
+ sources -= [ "rsa_private_key_nss_unittest.cc" ]
} else {
sources -= [ "openssl_bio_string_unittest.cc" ]
}
@@ -286,7 +295,7 @@ source_set("test_support") {
}
config("platform_config") {
- if (!use_openssl && is_clang) {
+ if ((!use_openssl || use_nss_certs) && is_clang) {
# There is a broken header guard in /usr/include/nss/secmod.h:
# https://bugzilla.mozilla.org/show_bug.cgi?id=884072
cflags = [ "-Wno-header-guard" ]
@@ -305,21 +314,26 @@ group("platform") {
deps = [
"//net/third_party/nss/ssl:libssl",
]
+ }
+
+ # Link in NSS if it is used for either the internal crypto library
+ # (!use_openssl) or platform certificate library (use_nss_certs).
+ if (!use_openssl || use_nss_certs) {
if (is_linux) {
# On Linux, we use the system NSS (excepting SSL where we always use our
# own).
- #
- # We always need our SSL header search path to come before the system one
- # so our versions are used. The libssl target will add the search path we
- # want, but according to GN's ordering rules, public_configs' search path
- # will get applied before ones inherited from our dependencies.
- # Therefore, we need to explicitly list our custom libssl's config here
- # before the system one.
- public_configs = [
- ":platform_config",
- "//net/third_party/nss/ssl:ssl_config",
- "//third_party/nss:system_nss_no_ssl_config",
- ]
+ public_configs = [ ":platform_config" ]
+ if (!use_openssl) {
+ # If using a bundled copy of NSS's SSL library, ensure the bundled SSL
+ # header search path comes before the system one so our versions are
+ # used. The libssl target will add the search path we want, but
+ # according to GN's ordering rules, public_configs' search path will get
+ # applied before ones inherited from our dependencies. Therefore, we
+ # need to explicitly list our custom libssl's config here before the
+ # system one.
+ public_configs += [ "//net/third_party/nss/ssl:ssl_config" ]
+ }
+ public_configs += [ "//third_party/nss:system_nss_no_ssl_config" ]
} else {
# Non-Linux platforms use the hermetic NSS from the tree.
deps += [
« no previous file with comments | « components/webcrypto/test/test_helpers.cc ('k') | crypto/crypto.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698