diff options
| -rw-r--r-- | data/fonts/Android.mk | 7 | ||||
| -rwxr-xr-x | tools/fonts/fontchain_lint.py | 16 |
2 files changed, 15 insertions, 8 deletions
diff --git a/data/fonts/Android.mk b/data/fonts/Android.mk index fd28f64db19e..e3639ecdeeb8 100644 --- a/data/fonts/Android.mk +++ b/data/fonts/Android.mk @@ -101,8 +101,13 @@ font_src_files := checkbuild: fontchain_lint FONTCHAIN_LINTER := frameworks/base/tools/fonts/fontchain_lint.py +ifeq ($(SMALLER_FONT_FOOTPRINT),true) +CHECK_EMOJI := false +else +CHECK_EMOJI := true +endif .PHONY: fontchain_lint fontchain_lint: $(FONTCHAIN_LINTER) $(TARGET_OUT)/etc/fonts.xml PYTHONPATH=$$PYTHONPATH:external/fonttools/Lib \ - python $(FONTCHAIN_LINTER) $(TARGET_OUT) external/unicode + python $(FONTCHAIN_LINTER) $(TARGET_OUT) $(CHECK_EMOJI) external/unicode diff --git a/tools/fonts/fontchain_lint.py b/tools/fonts/fontchain_lint.py index 75e837b43cfc..b5ed1b52a19d 100755 --- a/tools/fonts/fontchain_lint.py +++ b/tools/fonts/fontchain_lint.py @@ -177,9 +177,10 @@ def parse_fonts_xml(fonts_xml_path): def check_emoji_availability(): emoji_fonts = [font[5] for font in _fallback_chain if 'Zsye' in font[1]] + assert len(emoji_fonts) == 1, 'There are %d emoji fonts.' % len(emoji_fonts) + emoji_font = emoji_fonts[0] emoji_chars = _emoji_properties['Emoji'] - for emoji_font in emoji_fonts: - assert_font_supports_all_of_chars(emoji_font, emoji_chars) + assert_font_supports_all_of_chars(emoji_font, emoji_chars) def check_emoji_defaults(): @@ -273,11 +274,12 @@ def main(): hyphens_dir = path.join(target_out, 'usr', 'hyphen-data') check_hyphens(hyphens_dir) - ucd_path = sys.argv[2] - parse_ucd(ucd_path) - # Temporarily disable emoji checks for Bug 27785690 - # check_emoji_availability() - # check_emoji_defaults() + check_emoji = sys.argv[2] + if check_emoji == 'true': + ucd_path = sys.argv[3] + parse_ucd(ucd_path) + check_emoji_availability() + check_emoji_defaults() if __name__ == '__main__': |