Move conditionals inside target_defaults.

build/standalone.gypi

 # Copyright 2014 PDFium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # Definitions to be used when building stand-alone PDFium binaries.
 
 {
   'variables': {
     'component%': 'static_library',
     'clang%': 0,
@@ skipping 225 matching lines @@
       'OTHER_CFLAGS': [
         '-fno-strict-aliasing',
       ],
       'WARNING_CFLAGS': [
         '-Wall',
         '-Wendif-labels',
         '-W',
         '-Wno-unused-parameter',
       ],
     },
+    'conditions': [
+      ['component=="shared_library"', {

[Tom Sepez, 2015/10/15 20:26:56]

We'd want to this to apply to a release build as well.  Does that happen?

[Tom Sepez, 2015/10/15 20:38:14]

(I think it happens, but confirm).

[dsinclair, 2015/10/15 21:02:14]

Seems too, I'm able to complete 'ninja -C out/Release'.

+        'cflags': [
+          '-fPIC',
+        ],
+      }],
+      ['OS=="win"', {
+        'target_defaults': {

[Tom Sepez, 2015/10/15 20:25:50]

These target_defaults blocks may be redundant inside target_defaults.

[dsinclair, 2015/10/15 21:02:14]

Done.

+          'defines': [
+            'NOMINMAX',
+            '_CRT_SECURE_NO_DEPRECATE',
+            '_CRT_NONSTDC_NO_DEPRECATE',
+          ],
+          'conditions': [
+            ['component=="static_library"', {
+              'defines': [
+                '_HAS_EXCEPTIONS=0',
+              ],
+            }],
+          ],
+        },
+      }],  # OS=="win"
+      ['OS=="mac"', {
+        'target_defaults': {
+          'target_conditions': [

[Tom Sepez, 2015/10/15 20:25:50]

But this may have to remain target_conditions even after pulling the
target_defaults above.

[dsinclair, 2015/10/15 21:02:14]

Done.

+            ['_type!="static_library"', {
+              'xcode_settings': {'OTHER_LDFLAGS': ['-Wl,-search_paths_first']},
+            }],
+          ],  # target_conditions
+        },  # target_defaults
+      }],  # OS=="mac"
+      ['v8_use_external_startup_data==1', {
+        'target_defaults': {
+          'defines': [
+            'V8_USE_EXTERNAL_STARTUP_DATA',
+          ],
+        },
+      }],  # v8_use_external_startup_data==1
+    ],
   },
-  'conditions': [
-    ['component=="shared_library"', {
-      'cflags': [
-        '-fPIC',
-      ],
-    }],
-    ['OS=="win"', {
-      'target_defaults': {
-        'defines': [
-          'NOMINMAX',
-          '_CRT_SECURE_NO_DEPRECATE',
-          '_CRT_NONSTDC_NO_DEPRECATE',
-        ],
-        'conditions': [
-          ['component=="static_library"', {
-            'defines': [
-              '_HAS_EXCEPTIONS=0',
-            ],
-          }],
-        ],
-      },
-    }],  # OS=="win"
-    ['OS=="mac"', {
-      'target_defaults': {
-        'target_conditions': [
-          ['_type!="static_library"', {
-            'xcode_settings': {'OTHER_LDFLAGS': ['-Wl,-search_paths_first']},
-          }],
-        ],  # target_conditions
-      },  # target_defaults
-    }],  # OS=="mac"
-    ['v8_use_external_startup_data==1', {
-      'target_defaults': {
-        'defines': [
-          'V8_USE_EXTERNAL_STARTUP_DATA',
-        ],
-      },
-    }],  # v8_use_external_startup_data==1
-  ],
   'xcode_settings': {
     # See comment in Chromium's common.gypi for why this is needed.
     'SYMROOT': '<(DEPTH)/xcodebuild',
   }
 }