Replace most static_library entries with source_set.
Per GN documentation, "static_library: A .lib or .a file (normally
you'll want a source_set instead)."
Change-Id: If2ce44157f4d6c8998a0417f398f3bd19c052795
Reviewed-on: https://pdfium-review.googlesource.com/c/43893
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
diff --git a/BUILD.gn b/BUILD.gn
index d91c92b..b1293c2 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -240,7 +240,7 @@
}
}
-jumbo_static_library("test_support") {
+jumbo_source_set("test_support") {
testonly = true
sources = [
"testing/fx_string_testhelpers.cpp",
@@ -274,7 +274,7 @@
configs += [ ":pdfium_core_config" ]
}
-jumbo_static_library("image_diff") {
+jumbo_source_set("image_diff") {
testonly = true
sources = [
"testing/image_diff/image_diff_png.cpp",
@@ -302,7 +302,7 @@
]
}
-jumbo_static_library("fdrm") {
+jumbo_source_set("fdrm") {
sources = [
"core/fdrm/crypto/fx_crypt.cpp",
"core/fdrm/crypto/fx_crypt.h",
@@ -315,7 +315,7 @@
]
}
-jumbo_static_library("fpdfdoc") {
+jumbo_source_set("fpdfdoc") {
sources = [
"core/fpdfdoc/cline.cpp",
"core/fpdfdoc/cline.h",
@@ -398,7 +398,7 @@
]
}
-jumbo_static_library("fpdfapi") {
+jumbo_source_set("fpdfapi") {
sources = [
"core/fpdfapi/cmaps/CNS1/Adobe-CNS1-UCS2_5.cpp",
"core/fpdfapi/cmaps/CNS1/B5pc-H_0.cpp",
@@ -701,7 +701,7 @@
]
}
-jumbo_static_library("fpdftext") {
+jumbo_source_set("fpdftext") {
sources = [
"core/fpdftext/cpdf_linkextract.cpp",
"core/fpdftext/cpdf_linkextract.h",
@@ -718,7 +718,7 @@
]
}
-jumbo_static_library("fxcodec") {
+jumbo_source_set("fxcodec") {
sources = [
"core/fxcodec/JBig2_DocumentContext.h",
"core/fxcodec/codec/ccodec_basicmodule.h",
@@ -844,7 +844,7 @@
}
}
-jumbo_static_library("fxcrt") {
+jumbo_source_set("fxcrt") {
sources = [
"core/fxcrt/autorestorer.h",
"core/fxcrt/bytestring.cpp",
@@ -999,7 +999,7 @@
}
}
-jumbo_static_library("fxge") {
+jumbo_source_set("fxge") {
sources = [
"core/fxge/android/cfpf_skiadevicemodule.cpp",
"core/fxge/android/cfpf_skiadevicemodule.h",
@@ -1160,7 +1160,7 @@
}
}
-jumbo_static_library("pwl") {
+jumbo_source_set("pwl") {
sources = [
"fpdfsdk/pwl/cpwl_appstream.cpp",
"fpdfsdk/pwl/cpwl_appstream.h",
@@ -1201,7 +1201,7 @@
]
}
-jumbo_static_library("fxjs") {
+jumbo_source_set("fxjs") {
sources = [
"fxjs/cjs_event_context_stub.cpp",
"fxjs/cjs_event_context_stub.h",
@@ -1635,7 +1635,7 @@
}
}
-jumbo_static_library("formfiller") {
+jumbo_source_set("formfiller") {
sources = [
"fpdfsdk/formfiller/cba_fontmap.cpp",
"fpdfsdk/formfiller/cba_fontmap.h",
@@ -1667,7 +1667,7 @@
}
if (pdf_enable_xfa) {
- jumbo_static_library("fpdfxfa") {
+ jumbo_source_set("fpdfxfa") {
sources = [
"fpdfsdk/fpdfxfa/cpdfxfa_context.cpp",
"fpdfsdk/fpdfxfa/cpdfxfa_context.h",
@@ -1686,7 +1686,7 @@
configs += [ ":pdfium_core_config" ]
}
- jumbo_static_library("fxbarcode") {
+ jumbo_source_set("fxbarcode") {
sources = [
"fxbarcode/BC_Library.cpp",
"fxbarcode/BC_Library.h",
@@ -1831,7 +1831,7 @@
]
}
- jumbo_static_library("xfa_fde") {
+ jumbo_source_set("xfa_fde") {
sources = [
"xfa/fde/cfde_data.h",
"xfa/fde/cfde_texteditengine.cpp",
@@ -1850,7 +1850,7 @@
]
}
- jumbo_static_library("xfa_fgas") {
+ jumbo_source_set("xfa_fgas") {
sources = [
"xfa/fgas/crt/cfgas_formatstring.cpp",
"xfa/fgas/crt/cfgas_formatstring.h",
@@ -1893,7 +1893,7 @@
}
}
- jumbo_static_library("xfa_fwl") {
+ jumbo_source_set("xfa_fwl") {
sources = [
"xfa/fwl/cfwl_app.cpp",
"xfa/fwl/cfwl_app.h",
@@ -2017,7 +2017,7 @@
]
}
- jumbo_static_library("xfa_fxfa") {
+ jumbo_source_set("xfa_fxfa") {
sources = [
"xfa/fxfa/cxfa_eventparam.cpp",
"xfa/fxfa/cxfa_eventparam.h",
@@ -2120,7 +2120,7 @@
]
}
- jumbo_static_library("xfa_fxfa_fm2js") {
+ jumbo_source_set("xfa_fxfa_fm2js") {
sources = [
"xfa/fxfa/fm2js/cxfa_fmexpression.cpp",
"xfa/fxfa/fm2js/cxfa_fmexpression.h",
@@ -2142,7 +2142,7 @@
]
}
- jumbo_static_library("xfa_fxfa_parser") {
+ jumbo_source_set("xfa_fxfa_parser") {
sources = [
"xfa/fxfa/parser/cscript_datawindow.cpp",
"xfa/fxfa/parser/cscript_datawindow.h",
@@ -2846,7 +2846,7 @@
]
}
- jumbo_static_library("xfa_fxgraphics") {
+ jumbo_source_set("xfa_fxgraphics") {
sources = [
"xfa/fxgraphics/cxfa_gecolor.cpp",
"xfa/fxgraphics/cxfa_gecolor.h",
diff --git a/third_party/BUILD.gn b/third_party/BUILD.gn
index ed360a6..0430832 100644
--- a/third_party/BUILD.gn
+++ b/third_party/BUILD.gn
@@ -23,7 +23,7 @@
configs = [ "..:pdfium_common_config" ]
}
-static_library("bigint") {
+source_set("bigint") {
configs -= [ "//build/config/compiler:chromium_code" ]
configs += [
"//build/config/compiler:no_chromium_code",
@@ -77,7 +77,7 @@
# Tests may link against this even if the production library doesn't,
# so it needs to be separate from it.
- jumbo_static_library("fx_freetype") {
+ jumbo_source_set("fx_freetype") {
configs -= [ "//build/config/compiler:chromium_code" ]
configs += [
"//build/config/compiler:no_chromium_code",
@@ -200,7 +200,7 @@
}
}
- jumbo_static_library("fx_agg") {
+ jumbo_source_set("fx_agg") {
configs -= [ "//build/config/compiler:chromium_code" ]
configs += [
"//build/config/compiler:no_chromium_code",
@@ -246,7 +246,7 @@
}
}
-static_library("fx_lcms2") {
+source_set("fx_lcms2") {
configs -= [ "//build/config/compiler:chromium_code" ]
configs += [
"//build/config/compiler:no_chromium_code",
@@ -367,7 +367,7 @@
}
}
-static_library("fx_libopenjpeg") {
+source_set("fx_libopenjpeg") {
configs -= [ "//build/config/compiler:chromium_code" ]
configs += [
"//build/config/compiler:no_chromium_code",
@@ -417,7 +417,7 @@
}
}
-jumbo_static_library("fx_lpng") {
+jumbo_source_set("fx_lpng") {
configs -= [ "//build/config/compiler:chromium_code" ]
configs += [
"//build/config/compiler:no_chromium_code",
@@ -480,7 +480,7 @@
}
if (pdf_enable_xfa) {
- static_library("fx_tiff") {
+ source_set("fx_tiff") {
configs -= [ "//build/config/compiler:chromium_code" ]
configs += [
"//build/config/compiler:no_chromium_code",
