Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@jason-simmons
Copy link
Member

No description provided.

Copy link
Contributor

@chinmaygarde chinmaygarde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Edited away an old draft.

declare_args() {
flutter_use_fontconfig = false
}

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assert that you are on linux here as it is the only platform on which FC is supported.

@chinmaygarde
Copy link
Contributor

Presubmits are mad at the formatting of the GN files.

@jason-simmons
Copy link
Member Author

Done - PTAL

@jason-simmons jason-simmons merged commit 424940f into flutter:master May 16, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 16, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 16, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request May 17, 2019
flutter/engine@859cd98...e94d352

git log 859cd98..e94d352 --no-merges --oneline
e94d352 Roll src/third_party/dart cc2d5adec9..5ffff98440 (3 commits)
424940f Add support for the Fontconfig-based Skia font manager (flutter/engine#8977)

The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff ([email protected]), and stop
the roller if necessary.
huqiuser pushed a commit to huqiuser/engine that referenced this pull request Jun 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants