GYP changes to get sql and sqlite building on iOS.

third_party/sqlite/sqlite.gyp

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 {
   'variables': {
     'use_system_sqlite%': 0,
     'required_sqlite_version': '3.6.1',
   },
   'target_defaults': {
@@ skipping 25 matching lines @@
         ],
         ['OS=="linux" and not use_system_sqlite', {
           'link_settings': {
             'libraries': [
               '-ldl',
             ],
           },
         }],
         ['os_posix == 1 and OS != "mac" and use_system_sqlite', {
           'type': 'none',
+
           'direct_dependent_settings': {
-            'cflags': [
-              # This next command produces no output but it it will fail (and
-              # cause GYP to fail) if we don't have a recent enough version of
-              # sqlite.
-              '<!@(pkg-config --atleast-version=<(required_sqlite_version) sqlite3)',
-
-              '<!@(pkg-config --cflags sqlite3)',
-            ],
             'defines': [
               'USE_SYSTEM_SQLITE',
             ],
-          },
-          'link_settings': {
-            'ldflags': [
-              '<!@(pkg-config --libs-only-L --libs-only-other sqlite3)',
-            ],
-            'libraries': [
-              '<!@(pkg-config --libs-only-l sqlite3)',
+            'conditions': [
+              ['OS != "ios"', {
+                'cflags': [
+                  # This next command produces no output but it it will fail
+                  # (and cause GYP to fail) if we don't have a recent enough
+                  # version of sqlite.
+                  '<!@(pkg-config --atleast-version=<(required_sqlite_version) sqlite3)',
+                  '<!@(pkg-config --cflags sqlite3)',
+                ],
+              }],
             ],
           },
+
+
+          'conditions': [
+            ['OS == "ios"', {
+              'link_settings': {
+                 'libraries': [
+                   '$(SDKROOT)/usr/lib/libsqlite3.dylib',
+                 ],
+               },
+            },{  # OS != ios
+              'link_settings': {
+                'ldflags': [
+                  '<!@(pkg-config --libs-only-L --libs-only-other sqlite3)',
+                ],
+                'libraries': [
+                  '<!@(pkg-config --libs-only-l sqlite3)',
+                ],
+              },
+            }],
+          ],
         }, { # else: os_posix == 1 or OS == "mac" or ! use_system_sqlite

[stuartmorgan, 2012/07/13 12:18:10]

As long as you are here, s/1/0/ so this comment stops being a lie :)

Although, part of what makes this whole set of conditions so horrible is that it
makes no sense that if, say mac set use_system_sqlite=1, it would actually end
up in this path and silently just build from scratch. That's terrible.

How we structure the conditions in this target as
[ use_system_sqlite, {
  < direct_dependent_settings -> define USE_SYSTEM_SQLITE >
  'conditions': [
    [ OS == "ios", {
      < the ios-specific stuff >
    }],
    [ os_posix and OS != mac and OS != ios, {
      < the linux stuff >
    }],
  ]
}, {
  < everything that's currently below this line>
}]

That way it's clear that mac + use_system_sqlite isn't currently implemented.
Also, doing a direct_dependent_settings again in the Linux part to add cflags
avoids the confusing use of iOS checks in two different but similar places.

What do you think?

           'product_name': 'sqlite3',
           'type': 'static_library',
           'sources': [
             'amalgamation/sqlite3.h',
             'amalgamation/sqlite3.c',
             # fts2.c currently has a lot of conflicts when added to
             # the amalgamation.  It is probably not worth fixing that.
             'src/ext/fts2/fts2.c',
             'src/ext/fts2/fts2.h',
             'src/ext/fts2/fts2_hash.c',
@@ skipping 74 matching lines @@
             'src/src/shell_icu_linux.c',
           ],
           'link_settings': {
             'link_languages': ['c++'],
           },
         },
       ],
     },]
   ],
 }