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

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

Issue 1961823003: Remove SQLITE_NO_SYNC on Chrome OS (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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
« no previous file with comments | « third_party/sqlite/BUILD.gn ('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 '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 123 matching lines...) Expand 10 before | Expand all | Expand 10 after
134 'conditions': [ 134 'conditions': [
135 ['OS == "win" and component == "shared_library"', { 135 ['OS == "win" and component == "shared_library"', {
136 'defines': ['SQLITE_API=__declspec(dllexport)'], 136 'defines': ['SQLITE_API=__declspec(dllexport)'],
137 'direct_dependent_settings': { 137 'direct_dependent_settings': {
138 'defines': ['SQLITE_API=__declspec(dllimport)'], 138 'defines': ['SQLITE_API=__declspec(dllimport)'],
139 }, 139 },
140 }], 140 }],
141 ['OS != "win" and component == "shared_library"', { 141 ['OS != "win" and component == "shared_library"', {
142 'defines': ['SQLITE_API=__attribute__((visibility("default")))'], 142 'defines': ['SQLITE_API=__attribute__((visibility("default")))'],
143 }], 143 }],
144 [ 'chromeos==1' , {
145 'defines': [
146 # Despite obvious warnings about not using this flag in
147 # deployment, we are turning off sync in ChromeOS and relying on
148 # the underlying journaling filesystem to do error recovery
149 # properly. It's much faster.
150 'SQLITE_NO_SYNC',
151 ],
152 }],
153 ['os_posix == 1', { 144 ['os_posix == 1', {
154 'defines': [ 145 'defines': [
155 # Allow xSleep() call on Unix to use usleep() rather than 146 # Allow xSleep() call on Unix to use usleep() rather than
156 # sleep(). Microsecond precision is better than second 147 # sleep(). Microsecond precision is better than second
157 # precision. Should only affect contended databases via the 148 # precision. Should only affect contended databases via the
158 # busy callback. Browser profile databases are mostly 149 # busy callback. Browser profile databases are mostly
159 # exclusive, but renderer databases may allow for contention. 150 # exclusive, but renderer databases may allow for contention.
160 'HAVE_USLEEP=1', 151 'HAVE_USLEEP=1',
161 # Use pread/pwrite directly rather than emulating them. 152 # Use pread/pwrite directly rather than emulating them.
162 'USE_PREAD=1', 153 'USE_PREAD=1',
(...skipping 110 matching lines...) Expand 10 before | Expand all | Expand 10 after
273 'clang_warning_flags_unset': [ 264 'clang_warning_flags_unset': [
274 # icu.c uses assert(!"foo") instead of assert(false && "foo") 265 # icu.c uses assert(!"foo") instead of assert(false && "foo")
275 '-Wstring-conversion', 266 '-Wstring-conversion',
276 ], 267 ],
277 }, 268 },
278 }, 269 },
279 ], 270 ],
280 }], 271 }],
281 ], 272 ],
282 } 273 }
OLDNEW
« no previous file with comments | « third_party/sqlite/BUILD.gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698