XFA: Hide xfa targets in gyp (similar to GN) Allows "all" to include pdfium:* once again, since targets that don't exist except under xfa are hidden. R=thestig@chromium.org Review URL: https://codereview.chromium.org/1502343007 .
diff --git a/build/all.gyp b/build/all.gyp index 62db131..7e2ea61 100644 --- a/build/all.gyp +++ b/build/all.gyp
@@ -8,9 +8,7 @@ 'target_name': 'All', 'type': 'none', 'dependencies': [ - '../pdfium.gyp:pdfium', - '../pdfium.gyp:pdfium_embeddertests', - '../pdfium.gyp:pdfium_unittests', + '../pdfium.gyp:*', '../samples/samples.gyp:*', ], }
diff --git a/pdfium.gyp b/pdfium.gyp index 82a170d..934916f 100644 --- a/pdfium.gyp +++ b/pdfium.gyp
@@ -651,24 +651,6 @@ ], }, { - 'target_name': 'fpdfxfa', - 'type': 'static_library', - 'dependencies': [ - 'javascript', - 'xfa.gyp:xfa', - ], - 'sources': [ - 'fpdfsdk/src/fpdfxfa/fpdfxfa_app.cpp', - 'fpdfsdk/src/fpdfxfa/fpdfxfa_doc.cpp', - 'fpdfsdk/src/fpdfxfa/fpdfxfa_page.cpp', - 'fpdfsdk/src/fpdfxfa/fpdfxfa_util.cpp', - 'fpdfsdk/include/fpdfxfa/fpdfxfa_app.h', - 'fpdfsdk/include/fpdfxfa/fpdfxfa_doc.h', - 'fpdfsdk/include/fpdfxfa/fpdfxfa_page.h', - 'fpdfsdk/include/fpdfxfa/fpdfxfa_util.h', - ], - }, - { 'target_name': 'javascript', 'type': 'static_library', 'sources': [ @@ -875,4 +857,28 @@ ], }, ], + 'conditions': [ + ['pdf_enable_xfa==1', { + 'targets': [ + { + 'target_name': 'fpdfxfa', + 'type': 'static_library', + 'dependencies': [ + 'javascript', + 'xfa.gyp:xfa', + ], + 'sources': [ + 'fpdfsdk/src/fpdfxfa/fpdfxfa_app.cpp', + 'fpdfsdk/src/fpdfxfa/fpdfxfa_doc.cpp', + 'fpdfsdk/src/fpdfxfa/fpdfxfa_page.cpp', + 'fpdfsdk/src/fpdfxfa/fpdfxfa_util.cpp', + 'fpdfsdk/include/fpdfxfa/fpdfxfa_app.h', + 'fpdfsdk/include/fpdfxfa/fpdfxfa_doc.h', + 'fpdfsdk/include/fpdfxfa/fpdfxfa_page.h', + 'fpdfsdk/include/fpdfxfa/fpdfxfa_util.h', + ], + }, + ] + }], + ] }