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

Unified Diff: third_party/boringssl/boringssl_configurations.gypi

Issue 1924693003: Rolls BoringSSL forward (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 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 | « third_party/boringssl/boringssl.gypi ('k') | third_party/boringssl/boringssl_dart.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/boringssl/boringssl_configurations.gypi
diff --git a/third_party/boringssl/boringssl_configurations.gypi b/third_party/boringssl/boringssl_configurations.gypi
index b963505bc12aff180e9821624e82d6aa95745f77..9c49b0153b2a101f28571ca6c0b28f54fffa7f53 100644
--- a/third_party/boringssl/boringssl_configurations.gypi
+++ b/third_party/boringssl/boringssl_configurations.gypi
@@ -4,29 +4,10 @@
# This file is included to modify the configurations to build third-party
# code from BoringSSL.
-# This code is C code, not C++, and is not warning-free, so we need to remove
zra 2016/04/27 21:38:52 The code actually does appear to be warning-free.
-# C++-specific flags, and add flags to supress the warnings in the code.
{
- 'variables': {
zra 2016/04/27 21:38:52 These appear to be unused.
- # Used by third_party/nss, which is from Chromium.
- # Include the built-in set of root certificate authorities.
- 'exclude_nss_root_certs': 0,
- 'os_posix%': 1,
- 'os_bsd%': 0,
- 'chromeos%': 0,
- 'clang%': 0,
- },
'target_defaults': {
- 'cflags': [
- '-w',
- '-UHAVE_CVAR_BUILT_ON_SEM',
- ],
# Removes these flags from the list cflags.
'cflags!': [
- # NSS code from upstream mozilla builds with warnings,
- # so we must allow warnings without failing.
- '-Werror',
- '-Wall',
'-ansi',
# Not supported for C, only for C++.
'-Wnon-virtual-dtor',
@@ -35,93 +16,5 @@
'-fvisibility-inlines-hidden',
'-Woverloaded-virtual',
],
- 'configurations': {
- 'Dart_Base': {
- 'xcode_settings': {
- 'WARNING_CFLAGS': [
- '-w',
- ],
- 'WARNING_CFLAGS!': [
- '-Wall',
- '-Wextra',
- ],
- },
- },
- # Dart_Macos_Debug and Dart_Macos_Release are merged after
- # Dart_Macos_Base, so we can override the 'ansi' and '-Werror' flags set
- # at the global level in tools/gyp/configurations_xcode.gypi.
- 'Dart_Macos_Debug': {
zra 2016/04/27 21:38:52 We don't use boringssl on mac anymore.
- 'abstract': 1,
- 'xcode_settings': {
- # Remove 'ansi' setting.
- 'GCC_C_LANGUAGE_STANDARD': 'c99',
- 'GCC_TREAT_WARNINGS_AS_ERRORS': 'NO', # -Werror off
- },
- },
- 'Dart_Macos_Release': {
- 'abstract': 1,
- 'xcode_settings': {
- # Remove 'ansi' setting.
- 'GCC_C_LANGUAGE_STANDARD': 'c99',
- 'GCC_TREAT_WARNINGS_AS_ERRORS': 'NO', # -Werror off
- },
- },
- # Disable hand-coded assembly routines on ARMv6 and ARMv5TE.
zra 2016/04/27 21:38:52 The hand-coded assembly builds and runs on arm64,
- 'Dart_armv6_Base': {
- 'abstract': 1,
- 'defines': [
- 'OPENSSL_NO_ASM',
- ],
- },
- 'Dart_armv5te_Base': {
- 'abstract': 1,
- 'defines': [
- 'OPENSSL_NO_ASM',
- ],
- },
- # TODO(24321): Also disable temporarily on arm64. Reenable after the next
- # roll.
- 'Dart_arm64_Base': {
- 'abstract': 1,
- 'defines': [
- 'OPENSSL_NO_ASM',
- ],
- },
- # Android 64-bit dbc build is for arm64, disable temporarily as well.
- 'Dart_Android_arm64_Base': {
- 'abstract': 1,
- 'defines': [
- 'OPENSSL_NO_ASM',
- ],
- },
- # When being built for Android nss expects __linux__ to be defined.
- 'Dart_Android_Base': {
zra 2016/04/27 21:38:52 It seems this is no longer needed.
- 'target_conditions': [
- ['_toolset=="host"', {
- 'defines!': [
- 'ANDROID',
- ],
- # Define __linux__ on Android build for NSS.
- 'defines': [
- '__linux__',
- ],
- 'cflags!': [
- '-U__linux__',
- ],
- }],
- ['_toolset=="target"', {
- 'defines': [
- '__linux__',
- 'CHECK_FORK_GETPID', # Android does not provide pthread_atfork.
- '__USE_LARGEFILE64',
- ],
- # Define __linux__ on Android build for NSS.
- 'cflags!': [
- '-U__linux__',
- ],
- }]
- ],
- },
- },
},
}
« no previous file with comments | « third_party/boringssl/boringssl.gypi ('k') | third_party/boringssl/boringssl_dart.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698