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

Unified Diff: build/standalone.gypi

Issue 1407903002: Move conditionals inside target_defaults. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Created 5 years, 2 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/standalone.gypi
diff --git a/build/standalone.gypi b/build/standalone.gypi
index c0eaef419df30e2ff6538aed44e0940d87aa128d..507368dc25020967b4a0010a4f60db33a238ce4f 100644
--- a/build/standalone.gypi
+++ b/build/standalone.gypi
@@ -243,15 +243,13 @@
'-Wno-unused-parameter',
],
},
- },
- 'conditions': [
- ['component=="shared_library"', {
- 'cflags': [
- '-fPIC',
- ],
- }],
- ['OS=="win"', {
- 'target_defaults': {
+ 'conditions': [
+ ['component=="shared_library"', {
+ 'cflags': [
+ '-fPIC',
+ ],
+ }],
+ ['OS=="win"', {
'defines': [
'NOMINMAX',
'_CRT_SECURE_NO_DEPRECATE',
@@ -264,25 +262,21 @@
],
}],
],
- },
- }], # OS=="win"
- ['OS=="mac"', {
- 'target_defaults': {
+ }], # OS=="win"
+ ['OS=="mac"', {
'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': {
+ }], # OS=="mac"
+ ['v8_use_external_startup_data==1', {
'defines': [
'V8_USE_EXTERNAL_STARTUP_DATA',
],
- },
- }], # v8_use_external_startup_data==1
- ],
+ }], # v8_use_external_startup_data==1
+ ],
+ },
'xcode_settings': {
# See comment in Chromium's common.gypi for why this is needed.
'SYMROOT': '<(DEPTH)/xcodebuild',
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698