Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 # Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file | 1 # Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file |
| 2 # for details. All rights reserved. Use of this source code is governed by a | 2 # for details. All rights reserved. Use of this source code is governed by a |
| 3 # BSD-style license that can be found in the LICENSE file. | 3 # BSD-style license that can be found in the LICENSE file. |
| 4 | 4 |
| 5 # This file is included to modify the configurations to build third-party | 5 # This file is included to modify the configurations to build third-party |
| 6 # code from BoringSSL. | 6 # code from BoringSSL. |
| 7 # 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.
| |
| 8 # C++-specific flags, and add flags to supress the warnings in the code. | |
| 9 { | 7 { |
| 10 'variables': { | |
|
zra
2016/04/27 21:38:52
These appear to be unused.
| |
| 11 # Used by third_party/nss, which is from Chromium. | |
| 12 # Include the built-in set of root certificate authorities. | |
| 13 'exclude_nss_root_certs': 0, | |
| 14 'os_posix%': 1, | |
| 15 'os_bsd%': 0, | |
| 16 'chromeos%': 0, | |
| 17 'clang%': 0, | |
| 18 }, | |
| 19 'target_defaults': { | 8 'target_defaults': { |
| 20 'cflags': [ | |
| 21 '-w', | |
| 22 '-UHAVE_CVAR_BUILT_ON_SEM', | |
| 23 ], | |
| 24 # Removes these flags from the list cflags. | 9 # Removes these flags from the list cflags. |
| 25 'cflags!': [ | 10 'cflags!': [ |
| 26 # NSS code from upstream mozilla builds with warnings, | |
| 27 # so we must allow warnings without failing. | |
| 28 '-Werror', | |
| 29 '-Wall', | |
| 30 '-ansi', | 11 '-ansi', |
| 31 # Not supported for C, only for C++. | 12 # Not supported for C, only for C++. |
| 32 '-Wnon-virtual-dtor', | 13 '-Wnon-virtual-dtor', |
| 33 '-Wno-conversion-null', | 14 '-Wno-conversion-null', |
| 34 '-fno-rtti', | 15 '-fno-rtti', |
| 35 '-fvisibility-inlines-hidden', | 16 '-fvisibility-inlines-hidden', |
| 36 '-Woverloaded-virtual', | 17 '-Woverloaded-virtual', |
| 37 ], | 18 ], |
| 38 'configurations': { | |
| 39 'Dart_Base': { | |
| 40 'xcode_settings': { | |
| 41 'WARNING_CFLAGS': [ | |
| 42 '-w', | |
| 43 ], | |
| 44 'WARNING_CFLAGS!': [ | |
| 45 '-Wall', | |
| 46 '-Wextra', | |
| 47 ], | |
| 48 }, | |
| 49 }, | |
| 50 # Dart_Macos_Debug and Dart_Macos_Release are merged after | |
| 51 # Dart_Macos_Base, so we can override the 'ansi' and '-Werror' flags set | |
| 52 # at the global level in tools/gyp/configurations_xcode.gypi. | |
| 53 'Dart_Macos_Debug': { | |
|
zra
2016/04/27 21:38:52
We don't use boringssl on mac anymore.
| |
| 54 'abstract': 1, | |
| 55 'xcode_settings': { | |
| 56 # Remove 'ansi' setting. | |
| 57 'GCC_C_LANGUAGE_STANDARD': 'c99', | |
| 58 'GCC_TREAT_WARNINGS_AS_ERRORS': 'NO', # -Werror off | |
| 59 }, | |
| 60 }, | |
| 61 'Dart_Macos_Release': { | |
| 62 'abstract': 1, | |
| 63 'xcode_settings': { | |
| 64 # Remove 'ansi' setting. | |
| 65 'GCC_C_LANGUAGE_STANDARD': 'c99', | |
| 66 'GCC_TREAT_WARNINGS_AS_ERRORS': 'NO', # -Werror off | |
| 67 }, | |
| 68 }, | |
| 69 # 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,
| |
| 70 'Dart_armv6_Base': { | |
| 71 'abstract': 1, | |
| 72 'defines': [ | |
| 73 'OPENSSL_NO_ASM', | |
| 74 ], | |
| 75 }, | |
| 76 'Dart_armv5te_Base': { | |
| 77 'abstract': 1, | |
| 78 'defines': [ | |
| 79 'OPENSSL_NO_ASM', | |
| 80 ], | |
| 81 }, | |
| 82 # TODO(24321): Also disable temporarily on arm64. Reenable after the next | |
| 83 # roll. | |
| 84 'Dart_arm64_Base': { | |
| 85 'abstract': 1, | |
| 86 'defines': [ | |
| 87 'OPENSSL_NO_ASM', | |
| 88 ], | |
| 89 }, | |
| 90 # Android 64-bit dbc build is for arm64, disable temporarily as well. | |
| 91 'Dart_Android_arm64_Base': { | |
| 92 'abstract': 1, | |
| 93 'defines': [ | |
| 94 'OPENSSL_NO_ASM', | |
| 95 ], | |
| 96 }, | |
| 97 # When being built for Android nss expects __linux__ to be defined. | |
| 98 'Dart_Android_Base': { | |
|
zra
2016/04/27 21:38:52
It seems this is no longer needed.
| |
| 99 'target_conditions': [ | |
| 100 ['_toolset=="host"', { | |
| 101 'defines!': [ | |
| 102 'ANDROID', | |
| 103 ], | |
| 104 # Define __linux__ on Android build for NSS. | |
| 105 'defines': [ | |
| 106 '__linux__', | |
| 107 ], | |
| 108 'cflags!': [ | |
| 109 '-U__linux__', | |
| 110 ], | |
| 111 }], | |
| 112 ['_toolset=="target"', { | |
| 113 'defines': [ | |
| 114 '__linux__', | |
| 115 'CHECK_FORK_GETPID', # Android does not provide pthread_atfork. | |
| 116 '__USE_LARGEFILE64', | |
| 117 ], | |
| 118 # Define __linux__ on Android build for NSS. | |
| 119 'cflags!': [ | |
| 120 '-U__linux__', | |
| 121 ], | |
| 122 }] | |
| 123 ], | |
| 124 }, | |
| 125 }, | |
| 126 }, | 19 }, |
| 127 } | 20 } |
| OLD | NEW |