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

Side by Side Diff: third_party/zlib/zlib.gyp

Issue 437543007: Refactor how clang warning flags are set. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 6 years, 4 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
« third_party/snappy/snappy.gyp ('K') | « third_party/sqlite/sqlite.gyp ('k') | 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) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 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 { 5 {
6 'targets': [ 6 'targets': [
7 { 7 {
8 'target_name': 'zlib', 8 'target_name': 'zlib',
9 'type': 'static_library', 9 'type': 'static_library',
10 'sources': [ 10 'sources': [
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 ], 70 ],
71 'include_dirs': [ 71 'include_dirs': [
72 '.', 72 '.',
73 '../..', 73 '../..',
74 ], 74 ],
75 'direct_dependent_settings': { 75 'direct_dependent_settings': {
76 'include_dirs': [ 76 'include_dirs': [
77 '.', 77 '.',
78 ], 78 ],
79 }, 79 },
80 'variables': {
81 'clang_warning_flags': [
82 # zlib uses `if ((a == b))` for some reason.
83 '-Wno-parentheses-equality',
84 ],
85 },
80 'conditions': [ 86 'conditions': [
81 ['OS!="win"', { 87 ['OS!="win"', {
82 'sources!': [ 88 'sources!': [
83 'contrib/minizip/iowin32.c' 89 'contrib/minizip/iowin32.c'
84 ], 90 ],
85 }], 91 }],
86 ['OS=="android"', { 92 ['OS=="android"', {
87 'toolsets': ['target', 'host'], 93 'toolsets': ['target', 'host'],
88 }], 94 }],
89 ['OS=="mac" or OS=="ios" or os_bsd==1 or OS=="android"', { 95 ['OS=="mac" or OS=="ios" or os_bsd==1 or OS=="android"', {
90 # Mac, Android and the BSDs don't have fopen64, ftello64, or 96 # Mac, Android and the BSDs don't have fopen64, ftello64, or
91 # fseeko64. We use fopen, ftell, and fseek instead on these 97 # fseeko64. We use fopen, ftell, and fseek instead on these
92 # systems. 98 # systems.
93 'defines': [ 99 'defines': [
94 'USE_FILE32API' 100 'USE_FILE32API'
95 ], 101 ],
96 }], 102 }],
97 ['clang==1', {
98 'xcode_settings': {
99 'WARNING_CFLAGS': [
100 # zlib uses `if ((a == b))` for some reason.
101 '-Wno-parentheses-equality',
102 ],
103 },
104 'cflags': [
105 '-Wno-parentheses-equality',
106 ],
107 }],
108 ], 103 ],
109 }, 104 },
110 ], 105 ],
111 } 106 }
OLDNEW
« third_party/snappy/snappy.gyp ('K') | « third_party/sqlite/sqlite.gyp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698