win: get tools/crashpad_database_util mostly working

tools/tools.gyp

 # Copyright 2014 The Crashpad Authors. All rights reserved.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
 # You may obtain a copy of the License at
 #
 #     http://www.apache.org/licenses/LICENSE-2.0
 #
 # Unless required by applicable law or agreed to in writing, software
 # distributed under the License is distributed on an "AS IS" BASIS,
@@ skipping 28 matching lines @@
           ['GENERATOR=="ninja"', {
             'sectaskaccess_info_plist': '<!(pwd)/mac/sectaskaccess_info.plist',
           }, {  # else: GENERATOR!="ninja"
             'sectaskaccess_info_plist': 'mac/sectaskaccess_info.plist',
           }],
         ],
       },
 
       'targets': [
         {
-          'target_name': 'crashpad_tool_support',
-          'type': 'static_library',
-          'dependencies': [
-            '../third_party/mini_chromium/mini_chromium.gyp:base',
-          ],
-          'include_dirs': [
-            '..',
-          ],
-          'sources': [
-            'tool_support.cc',
-            'tool_support.h',
-          ],
-        },
-        {
-          'target_name': 'crashpad_database_util',
-          'type': 'executable',
-          'dependencies': [
-            'crashpad_tool_support',
-            '../client/client.gyp:crashpad_client',
-            '../compat/compat.gyp:crashpad_compat',
-            '../third_party/mini_chromium/mini_chromium.gyp:base',
-            '../util/util.gyp:crashpad_util',
-          ],
-          'include_dirs': [
-            '..',
-          ],
-          'sources': [
-            'crashpad_database_util.cc',
-          ],
-        },
-        {
           'target_name': 'catch_exception_tool',
           'type': 'executable',
           'dependencies': [
             'crashpad_tool_support',
             '../compat/compat.gyp:crashpad_compat',
             '../third_party/mini_chromium/mini_chromium.gyp:base',
             '../util/util.gyp:crashpad_util',
           ],
           'include_dirs': [
             '..',
@@ skipping 86 matching lines @@
           ],
           'include_dirs': [
             '..',
           ],
           'sources': [
             'mac/run_with_crashpad.cc',
           ],
         },
       ],
     }, {
-      'targets': [],
+      'targets': [

[Mark Mentovai, 2015/05/04 21:04:30]

Now this is in an else block that’s live on non-Mac, but we want it to be live
all the time. Let’s put a top-level 'targets' above the 'conditions' for this.

[scottmg, 2015/05/04 23:26:36]

Oh, geez, oops. Done.

+        {
+          'target_name': 'crashpad_tool_support',
+          'type': 'static_library',
+          'dependencies': [
+            '../third_party/mini_chromium/mini_chromium.gyp:base',
+          ],
+          'include_dirs': [
+            '..',
+          ],
+          'sources': [
+            'tool_support.cc',
+            'tool_support.h',
+          ],
+        },
+        {
+          'target_name': 'crashpad_database_util',
+          'type': 'executable',
+          'dependencies': [
+            'crashpad_tool_support',
+            '../client/client.gyp:crashpad_client',
+            '../compat/compat.gyp:crashpad_compat',
+            '../third_party/mini_chromium/mini_chromium.gyp:base',
+            '../util/util.gyp:crashpad_util',
+          ],
+          'include_dirs': [
+            '..',
+          ],
+          'sources': [
+            'crashpad_database_util.cc',
+          ],
+        },
+      ],
     }],
   ],
 }