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

Unified Diff: build/config/freetype/BUILD.gn

Issue 2863063003: Statically link own version of FreeType on Linux (Closed)
Patch Set: Rely on ChromeOS using the use_system_freetype switch Created 3 years, 7 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 | build/config/linux/pangocairo/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/config/freetype/BUILD.gn
diff --git a/build/config/freetype/BUILD.gn b/build/config/freetype/BUILD.gn
index 4f052646cd6b72544e9a3fe71262039f45def3c3..976661a7cb84a80ee949f141fc95efdf711a20d1 100644
--- a/build/config/freetype/BUILD.gn
+++ b/build/config/freetype/BUILD.gn
@@ -4,8 +4,19 @@
import("//build/config/features.gni")
+declare_args() {
+ # Blink needs a recent and properly build-configured FreeType version to
+ # support OpenType variations, color emoji and avoid security bugs. By default
+ # we ship and link such a version as part of Chrome. For distributions that
+ # prefer to keep linking to the version the system, FreeType must be newer
+ # than version 2.7.1 and have color bitmap support compiled in. WARNING:
+ # System FreeType configurations other than as described WILL INTRODUCE TEXT
+ # RENDERING AND SECURITY REGRESSIONS.
+ use_system_freetype = false
+}
+
group("freetype") {
- if (is_linux && !is_chromecast) {
+ if (use_system_freetype) {
public_configs = [ "//build/linux:freetype_from_pkgconfig" ]
} else {
public_deps = [
« no previous file with comments | « no previous file | build/config/linux/pangocairo/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698