From 9319c8b537460d193c09ba3687ae776cd28c990e Mon Sep 17 00:00:00 2001 From: "gspencer@chromium.org" Date: Tue, 22 Feb 2011 20:16:58 +0000 Subject: Add SQLITE_NO_SYNC define for ChromeOS build Despite obvious warnings about not using this flag in deployment, we are turning off sqlite sync in ChromeOS and relying on the underlying journaling filesystem to do error recovery properly. It's much faster. BUG=chromium-os:12328 TEST=Build both with and without SQLITE_NO_SYNC, and measured power usage at the SATA rail. Power reduction is about 2-3%, and light use so far produced no data corruption. Review URL: http://codereview.chromium.org/6544025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75623 0039d316-1c4b-4281-b951-d872f2087c98 --- third_party/sqlite/sqlite.gyp | 10 ++++++++++ 1 file changed, 10 insertions(+) (limited to 'third_party/sqlite') diff --git a/third_party/sqlite/sqlite.gyp b/third_party/sqlite/sqlite.gyp index dae9b13..71a66bd 100644 --- a/third_party/sqlite/sqlite.gyp +++ b/third_party/sqlite/sqlite.gyp @@ -24,6 +24,16 @@ { 'target_name': 'sqlite', 'conditions': [ + [ 'chromeos==1' , { + 'defines': [ + # Despite obvious warnings about not using this flag + # in deployment, we are turning off sync in ChromeOS + # and relying on the underlying journaling filesystem + # to do error recovery properly. It's much faster. + 'SQLITE_NO_SYNC', + ], + }, + ], ['OS=="linux" and not use_system_sqlite', { 'link_settings': { 'libraries': [ -- cgit v1.1