From 81a1ac47e5d93980b59b1c34e8cf1bb22a89a936 Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Thu, 21 Dec 2017 21:49:55 -0800 Subject: Add initial jumbo build support. Convert a bunch of targets that just work. BUG=pdfium:964 Change-Id: I485be45e9f6c74c2bba88313da176a49b32b9bba Reviewed-on: https://pdfium-review.googlesource.com/21970 Reviewed-by: dsinclair Commit-Queue: Lei Zhang --- BUILD.gn | 26 +++++++++++++++++--------- third_party/BUILD.gn | 9 +++++---- 2 files changed, 22 insertions(+), 13 deletions(-) diff --git a/BUILD.gn b/BUILD.gn index 50f83c8dbf..b5a8098c2c 100644 --- a/BUILD.gn +++ b/BUILD.gn @@ -3,6 +3,7 @@ # found in the LICENSE file. import("//build/config/freetype/freetype.gni") +import("//build/config/jumbo.gni") import("//testing/test.gni") import("pdfium.gni") @@ -100,7 +101,7 @@ config("xfa_warnings") { } } -static_library("pdfium") { +jumbo_static_library("pdfium") { sources = [ "fpdfsdk/cba_annotiterator.cpp", "fpdfsdk/cba_annotiterator.h", @@ -231,7 +232,7 @@ static_library("pdfium") { } } -static_library("test_support") { +jumbo_static_library("test_support") { testonly = true sources = [ "testing/fx_string_testhelpers.cpp", @@ -264,7 +265,7 @@ static_library("test_support") { configs += [ ":pdfium_core_config" ] } -static_library("image_diff") { +jumbo_static_library("image_diff") { testonly = true sources = [ "testing/image_diff/image_diff_png.cpp", @@ -284,6 +285,7 @@ visibility = [ "//:gn_visibility", ] +# TODO(crbug.com/pdfium/964): Support jumbo builds. static_library("fdrm") { sources = [ "core/fdrm/crypto/fx_crypt.cpp", @@ -297,6 +299,7 @@ static_library("fdrm") { ] } +# TODO(crbug.com/pdfium/964): Support jumbo builds. static_library("fpdfdoc") { sources = [ "core/fpdfdoc/cline.cpp", @@ -380,7 +383,7 @@ static_library("fpdfdoc") { ] } -static_library("fpdfapi") { +jumbo_static_library("fpdfapi") { sources = [ "core/fpdfapi/cmaps/CNS1/Adobe-CNS1-UCS2_5.cpp", "core/fpdfapi/cmaps/CNS1/B5pc-H_0.cpp", @@ -672,7 +675,7 @@ static_library("fpdfapi") { ] } -static_library("fpdftext") { +jumbo_static_library("fpdftext") { sources = [ "core/fpdftext/cpdf_linkextract.cpp", "core/fpdftext/cpdf_linkextract.h", @@ -689,7 +692,7 @@ static_library("fpdftext") { ] } -static_library("fxcodec") { +jumbo_static_library("fxcodec") { sources = [ "core/fxcodec/JBig2_DocumentContext.h", "core/fxcodec/codec/ccodec_basicmodule.h", @@ -801,6 +804,7 @@ config("fxge_warnings") { } } +# TODO(crbug.com/pdfium/964): Support jumbo builds. static_library("fxcrt") { sources = [ "core/fxcrt/autorestorer.h", @@ -967,6 +971,7 @@ static_library("fxcrt") { } } +# TODO(crbug.com/pdfium/964): Support jumbo builds. static_library("fxge") { sources = [ "core/fxge/android/cfpf_skiabufferfont.cpp", @@ -1137,6 +1142,7 @@ static_library("fxge") { } } +# TODO(crbug.com/pdfium/964): Support jumbo builds. static_library("pwl") { sources = [ "fpdfsdk/pwl/cpwl_appstream.cpp", @@ -1178,7 +1184,7 @@ static_library("pwl") { ] } -static_library("fxjs") { +jumbo_static_library("fxjs") { sources = [ "fxjs/ijs_event_context.h", "fxjs/ijs_runtime.h", @@ -1617,7 +1623,7 @@ static_library("fxjs") { } } -static_library("formfiller") { +jumbo_static_library("formfiller") { sources = [ "fpdfsdk/formfiller/cba_fontmap.cpp", "fpdfsdk/formfiller/cba_fontmap.h", @@ -1649,7 +1655,7 @@ static_library("formfiller") { } if (pdf_enable_xfa) { - static_library("fpdfxfa") { + jumbo_static_library("fpdfxfa") { sources = [ "fpdfsdk/fpdfxfa/cpdfxfa_context.cpp", "fpdfsdk/fpdfxfa/cpdfxfa_context.h", @@ -1668,6 +1674,7 @@ if (pdf_enable_xfa) { configs += [ ":pdfium_core_config" ] } + # TODO(crbug.com/pdfium/964): Support jumbo builds. static_library("fxbarcode") { sources = [ "fxbarcode/BC_Library.cpp", @@ -1805,6 +1812,7 @@ if (pdf_enable_xfa) { configs += [ ":pdfium_core_config" ] } + # TODO(crbug.com/pdfium/964): Support jumbo builds. static_library("xfa") { sources = [ "xfa/fde/cfde_texteditengine.cpp", diff --git a/third_party/BUILD.gn b/third_party/BUILD.gn index d9f7670bb5..1cfee77c60 100644 --- a/third_party/BUILD.gn +++ b/third_party/BUILD.gn @@ -3,6 +3,7 @@ # found in the LICENSE file. import("//build/config/arm.gni") +import("//build/config/jumbo.gni") import("//build/config/linux/pkg_config.gni") import("//build_overrides/build.gni") import("../pdfium.gni") @@ -75,7 +76,7 @@ if (pdf_bundle_freetype) { # Tests may link against this even if the production library doesn't, # so it needs to be separate from it. - static_library("fx_freetype") { + jumbo_static_library("fx_freetype") { configs -= [ "//build/config/compiler:chromium_code" ] configs += [ "//build/config/compiler:no_chromium_code", @@ -199,7 +200,7 @@ if (!pdf_use_skia && !pdf_use_skia_paths) { } } - static_library("fx_agg") { + jumbo_static_library("fx_agg") { configs -= [ "//build/config/compiler:chromium_code" ] configs += [ "//build/config/compiler:no_chromium_code", @@ -415,7 +416,7 @@ group("png") { } } -static_library("fx_lpng") { +jumbo_static_library("fx_lpng") { configs -= [ "//build/config/compiler:chromium_code" ] configs += [ "//build/config/compiler:no_chromium_code", @@ -534,7 +535,7 @@ if (pdf_enable_xfa) { } } -source_set("pdfium_base") { +jumbo_source_set("pdfium_base") { configs -= [ "//build/config/compiler:chromium_code" ] configs += [ "//build/config/compiler:no_chromium_code", -- cgit v1.2.3