Chromium Code Reviews| 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__', |
| - ], |
| - }] |
| - ], |
| - }, |
| - }, |
| }, |
| } |