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

Side by Side Diff: third_party/sqlite/sqlite.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
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 'variables': { 6 'variables': {
7 'use_system_sqlite%': 0, 7 'use_system_sqlite%': 0,
8 'required_sqlite_version': '3.6.1', 8 'required_sqlite_version': '3.6.1',
9 }, 9 },
10 'target_defaults': { 10 'target_defaults': {
(...skipping 101 matching lines...) Expand 10 before | Expand all | Expand 10 after
112 ], 112 ],
113 'direct_dependent_settings': { 113 'direct_dependent_settings': {
114 'include_dirs': [ 114 'include_dirs': [
115 '.', 115 '.',
116 '../..', 116 '../..',
117 ], 117 ],
118 }, 118 },
119 'msvs_disabled_warnings': [ 119 'msvs_disabled_warnings': [
120 4018, 4244, 4267, 120 4018, 4244, 4267,
121 ], 121 ],
122 'variables': {
123 'clang_warning_flags': [
124 # sqlite does `if (*a++ && *b++);` in a non-buggy way.
125 '-Wno-empty-body',
126 # sqlite has some `unsigned < 0` checks.
127 '-Wno-tautological-compare',
128 ],
129 },
122 'conditions': [ 130 'conditions': [
123 ['OS=="linux"', { 131 ['OS=="linux"', {
124 'link_settings': { 132 'link_settings': {
125 'libraries': [ 133 'libraries': [
126 '-ldl', 134 '-ldl',
127 ], 135 ],
128 }, 136 },
129 }], 137 }],
130 ['OS == "mac" or OS == "ios"', { 138 ['OS == "mac" or OS == "ios"', {
131 'link_settings': { 139 'link_settings': {
(...skipping 14 matching lines...) Expand all
146 }], 154 }],
147 ['os_posix == 1 and OS != "mac" and OS != "android"', { 155 ['os_posix == 1 and OS != "mac" and OS != "android"', {
148 'cflags': [ 156 'cflags': [
149 # SQLite doesn't believe in compiler warnings, 157 # SQLite doesn't believe in compiler warnings,
150 # preferring testing. 158 # preferring testing.
151 # http://www.sqlite.org/faq.html#q17 159 # http://www.sqlite.org/faq.html#q17
152 '-Wno-int-to-pointer-cast', 160 '-Wno-int-to-pointer-cast',
153 '-Wno-pointer-to-int-cast', 161 '-Wno-pointer-to-int-cast',
154 ], 162 ],
155 }], 163 }],
156 ['clang==1', {
157 'xcode_settings': {
158 'WARNING_CFLAGS': [
159 # sqlite does `if (*a++ && *b++);` in a non-buggy way.
160 '-Wno-empty-body',
161 # sqlite has some `unsigned < 0` checks.
162 '-Wno-tautological-compare',
163 ],
164 },
165 'cflags': [
166 '-Wno-empty-body',
167 '-Wno-tautological-compare',
168 ],
169 }],
170 ], 164 ],
171 }], 165 }],
172 ], 166 ],
173 }, 167 },
174 ], 168 ],
175 'conditions': [ 169 'conditions': [
176 ['os_posix == 1 and OS != "mac" and OS != "ios" and OS != "android" and not use_system_sqlite', { 170 ['os_posix == 1 and OS != "mac" and OS != "ios" and OS != "android" and not use_system_sqlite', {
177 'targets': [ 171 'targets': [
178 { 172 {
179 'target_name': 'sqlite_shell', 173 'target_name': 'sqlite_shell',
(...skipping 21 matching lines...) Expand all
201 '../icu/icu.gyp:icuuc', 195 '../icu/icu.gyp:icuuc',
202 ], 196 ],
203 'sources': [ 197 'sources': [
204 'src/ext/icu/icu.c', 198 'src/ext/icu/icu.c',
205 ], 199 ],
206 }, 200 },
207 ], 201 ],
208 }], 202 }],
209 ], 203 ],
210 } 204 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698