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

Side by Side Diff: build/common.gypi

Issue 3756003: Revert 62529 - Add -Wno-non-virtual-dtor flag on posix targets.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 10 years, 2 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 # IMPORTANT: 5 # IMPORTANT:
6 # Please don't directly include this file if you are building via gyp_chromium, 6 # Please don't directly include this file if you are building via gyp_chromium,
7 # since gyp_chromium is automatically forcing its inclusion. 7 # since gyp_chromium is automatically forcing its inclusion.
8 { 8 {
9 'variables': { 9 'variables': {
10 # .gyp files or targets should set chromium_code to 1 if they build 10 # .gyp files or targets should set chromium_code to 1 if they build
(...skipping 856 matching lines...) Expand 10 before | Expand all | Expand 10 after
867 # Note: this is *required* to make some plugins work. 867 # Note: this is *required* to make some plugins work.
868 '-fvisibility=hidden', 868 '-fvisibility=hidden',
869 '-pipe', 869 '-pipe',
870 ], 870 ],
871 'cflags_cc': [ 871 'cflags_cc': [
872 '-fno-rtti', 872 '-fno-rtti',
873 '-fno-threadsafe-statics', 873 '-fno-threadsafe-statics',
874 # Make inline functions have hidden visiblity by default. 874 # Make inline functions have hidden visiblity by default.
875 # Surprisingly, not covered by -fvisibility=hidden. 875 # Surprisingly, not covered by -fvisibility=hidden.
876 '-fvisibility-inlines-hidden', 876 '-fvisibility-inlines-hidden',
877 '-Wno-non-virtual-dtor',
878 ], 877 ],
879 'ldflags': [ 878 'ldflags': [
880 '-pthread', '-Wl,-z,noexecstack', 879 '-pthread', '-Wl,-z,noexecstack',
881 ], 880 ],
882 'scons_variable_settings': { 881 'scons_variable_settings': {
883 'LIBPATH': ['$LIB_DIR'], 882 'LIBPATH': ['$LIB_DIR'],
884 # Linking of large files uses lots of RAM, so serialize links 883 # Linking of large files uses lots of RAM, so serialize links
885 # using the handy flock command from util-linux. 884 # using the handy flock command from util-linux.
886 'FLOCK_LINK': ['flock', '$TOP_BUILDDIR/linker.lock', '$LINK'], 885 'FLOCK_LINK': ['flock', '$TOP_BUILDDIR/linker.lock', '$LINK'],
887 'FLOCK_SHLINK': ['flock', '$TOP_BUILDDIR/linker.lock', '$SHLINK'], 886 'FLOCK_SHLINK': ['flock', '$TOP_BUILDDIR/linker.lock', '$SHLINK'],
(...skipping 343 matching lines...) Expand 10 before | Expand all | Expand 10 after
1231 'GCC_ENABLE_CPP_RTTI': 'NO', # -fno-rtti 1230 'GCC_ENABLE_CPP_RTTI': 'NO', # -fno-rtti
1232 'GCC_ENABLE_PASCAL_STRINGS': 'NO', # No -mpascal-strings 1231 'GCC_ENABLE_PASCAL_STRINGS': 'NO', # No -mpascal-strings
1233 # GCC_INLINES_ARE_PRIVATE_EXTERN maps to -fvisibility-inlines-hidden 1232 # GCC_INLINES_ARE_PRIVATE_EXTERN maps to -fvisibility-inlines-hidden
1234 'GCC_INLINES_ARE_PRIVATE_EXTERN': 'YES', 1233 'GCC_INLINES_ARE_PRIVATE_EXTERN': 'YES',
1235 'GCC_OBJC_CALL_CXX_CDTORS': 'YES', # -fobjc-call-cxx-cdtors 1234 'GCC_OBJC_CALL_CXX_CDTORS': 'YES', # -fobjc-call-cxx-cdtors
1236 'GCC_SYMBOLS_PRIVATE_EXTERN': 'YES', # -fvisibility=hidden 1235 'GCC_SYMBOLS_PRIVATE_EXTERN': 'YES', # -fvisibility=hidden
1237 'GCC_THREADSAFE_STATICS': 'NO', # -fno-threadsafe-statics 1236 'GCC_THREADSAFE_STATICS': 'NO', # -fno-threadsafe-statics
1238 'GCC_TREAT_WARNINGS_AS_ERRORS': 'YES', # -Werror 1237 'GCC_TREAT_WARNINGS_AS_ERRORS': 'YES', # -Werror
1239 'GCC_VERSION': '4.2', 1238 'GCC_VERSION': '4.2',
1240 'GCC_WARN_ABOUT_MISSING_NEWLINE': 'YES', # -Wnewline-eof 1239 'GCC_WARN_ABOUT_MISSING_NEWLINE': 'YES', # -Wnewline-eof
1241 # TODO(cbentzel): http://crbug.com/45135
1242 # Enable non virtual destructor warning on OSX.
1243 # It's gcc implementation currently warns about more violations
1244 # than Linux's gcc implementation.
1245 #'GCC_WARN_NON_VIRTUAL_DESTRUCTOR': 'YES', # -Wno-non-virtual-dtor
1246 # MACOSX_DEPLOYMENT_TARGET maps to -mmacosx-version-min 1240 # MACOSX_DEPLOYMENT_TARGET maps to -mmacosx-version-min
1247 'MACOSX_DEPLOYMENT_TARGET': '<(mac_deployment_target)', 1241 'MACOSX_DEPLOYMENT_TARGET': '<(mac_deployment_target)',
1248 'PREBINDING': 'NO', # No -Wl,-prebind 1242 'PREBINDING': 'NO', # No -Wl,-prebind
1249 'USE_HEADERMAP': 'NO', 1243 'USE_HEADERMAP': 'NO',
1250 'WARNING_CFLAGS': [ 1244 'WARNING_CFLAGS': [
1251 '-Wall', 1245 '-Wall',
1252 '-Wendif-labels', 1246 '-Wendif-labels',
1253 '-Wextra', 1247 '-Wextra',
1254 # Don't warn about unused function parameters. 1248 # Don't warn about unused function parameters.
1255 '-Wno-unused-parameter', 1249 '-Wno-unused-parameter',
(...skipping 261 matching lines...) Expand 10 before | Expand all | Expand 10 after
1517 # and therefore SYMROOT, needs to be set at the project level. 1511 # and therefore SYMROOT, needs to be set at the project level.
1518 'SYMROOT': '<(DEPTH)/xcodebuild', 1512 'SYMROOT': '<(DEPTH)/xcodebuild',
1519 }, 1513 },
1520 } 1514 }
1521 1515
1522 # Local Variables: 1516 # Local Variables:
1523 # tab-width:2 1517 # tab-width:2
1524 # indent-tabs-mode:nil 1518 # indent-tabs-mode:nil
1525 # End: 1519 # End:
1526 # vim: set expandtab tabstop=2 shiftwidth=2: 1520 # vim: set expandtab tabstop=2 shiftwidth=2:
OLDNEW
« 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