diff options
author | Andrew Morrow <acm@10gen.com> | 2013-09-18 15:55:35 -0400 |
---|---|---|
committer | Andrew Morrow <acm@10gen.com> | 2013-09-18 17:38:57 -0400 |
commit | be4bc7d410dce6d82015143964c6f9bf76b16eec (patch) | |
tree | a607148b2b562618479a1c2c69e35ad79c1d26ff /src/third_party | |
parent | ef2647b519e5ed3912ca17c2107e830de7c621ce (diff) | |
download | mongo-be4bc7d410dce6d82015143964c6f9bf76b16eec.tar.gz |
SERVER-10726 Suppress warnings in build of internal v8
Diffstat (limited to 'src/third_party')
-rw-r--r-- | src/third_party/v8/SConscript | 17 |
1 files changed, 13 insertions, 4 deletions
diff --git a/src/third_party/v8/SConscript b/src/third_party/v8/SConscript index f2ef44d60d4..a464b5b942a 100644 --- a/src/third_party/v8/SConscript +++ b/src/third_party/v8/SConscript @@ -46,10 +46,7 @@ LIBRARY_FLAGS = { }, 'gcc': { 'all': { - 'CCFLAGS': ['-Wall', - '-Werror', - '-W', - '-Wno-unused-parameter', + 'CCFLAGS': ['-Wno-unused-parameter', '-Woverloaded-virtual', '-Wnon-virtual-dtor'] }, @@ -368,6 +365,18 @@ cppdefines = get_flags('CPPDEFINES', toolchain, options) cppdefines = filter(lambda f : f not in env['CPPDEFINES'], cppdefines) env.Append(CPPDEFINES=cppdefines) +# NOTE: Suppress attempts to enable warnings in v8. Trying to individually suppress with -Wno- +# results in a game of whack-a-mole between various versions of clang and gcc as they add new +# warnings. We won't be changing the v8 sources, so the warnings aren't helpful. +def removeIfPresent(lst, item): + try: + lst.remove(item) + except ValueError: + pass + +for to_remove in ['-Werror', '-Wall', '-W']: + removeIfPresent(env['CCFLAGS'], to_remove) + # specify rules for building libraries.cc and experimental-libraries.cc env['BUILDERS']['JS2C'] = Builder(action=js2c.JS2C) experimental_library_files = [('src/'+s) for s in EXPERIMENTAL_LIBRARY_FILES] |