Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: common.gypi

Issue 29377871: Issue 4950 - Add -fpermissive to allow building on recent gcc.This is caused by V8 code.Possibly… (Closed) Base URL: https://hg.adblockplus.org/libadblockplus/
Patch Set: Added comment as requested. Created March 6, 2017, 5:57 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« 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 { 1 {
2 'variables': { 2 'variables': {
3 'visibility%': 'hidden', 3 'visibility%': 'hidden',
4 'library%': 'static_library', 4 'library%': 'static_library',
5 'component%': '', 5 'component%': '',
6 'want_separate_host_toolset': 0, 6 'want_separate_host_toolset': 0,
7 'v8_optimized_debug': 0, 7 'v8_optimized_debug': 0,
8 'v8_enable_i18n_support': 0, 8 'v8_enable_i18n_support': 0,
9 }, 9 },
10 10
11 'conditions': [ 11 'conditions': [
12 ['OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="solaris" \ 12 ['OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="solaris" \
13 or OS=="netbsd" or OS=="android"', { 13 or OS=="netbsd" or OS=="android"', {
14 'target_defaults': { 14 'target_defaults': {
15 # remove the -fpermissive when a newer v8 is used
16 # unless it is still required.
15 'cflags': [ '-Wall', '-W', '-Wno-unused-parameter', 17 'cflags': [ '-Wall', '-W', '-Wno-unused-parameter',
16 '-Wnon-virtual-dtor', '-pthread', '-fno-rtti', 18 '-Wnon-virtual-dtor', '-pthread', '-fno-rtti',
17 '-pedantic', '-std=c++0x', '-fexceptions', ], 19 '-pedantic', '-std=c++0x', '-fexceptions', '-fpermissive' ],
18 'cflags!': [ '-Werror', ], 20 'cflags!': [ '-Werror', ],
19 'ldflags': [ '-pthread', ], 21 'ldflags': [ '-pthread', ],
20 }, 22 },
21 }], 23 }],
22 ['OS=="mac"', { 24 ['OS=="mac"', {
23 'xcode_settings': { 25 'xcode_settings': {
24 'CLANG_CXX_LANGUAGE_STANDARD': 'c++11', 26 'CLANG_CXX_LANGUAGE_STANDARD': 'c++11',
25 'CLANG_CXX_LIBRARY': 'libc++', 27 'CLANG_CXX_LIBRARY': 'libc++',
26 'OTHER_CPLUSPLUSFLAGS' : ['-std=c++11', '-stdlib=libc++'], 28 'OTHER_CPLUSPLUSFLAGS' : ['-std=c++11', '-stdlib=libc++'],
27 }, 29 },
(...skipping 14 matching lines...) Expand all
42 'msvs_cygwin_shell': 0, 44 'msvs_cygwin_shell': 0,
43 'target_conditions': [[ 45 'target_conditions': [[
44 'OS=="mac" and _type=="executable"', { 46 'OS=="mac" and _type=="executable"', {
45 'xcode_settings': { 47 'xcode_settings': {
46 'OTHER_LDFLAGS': ['-stdlib=libc++'], 48 'OTHER_LDFLAGS': ['-stdlib=libc++'],
47 }, 49 },
48 } 50 }
49 ]], 51 ]],
50 } 52 }
51 } 53 }
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