diff --git a/CHANGELOG.md b/CHANGELOG.md index 729ea3912d..d3f2852e6a 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -26,6 +26,7 @@ A more detailed list of changes is available in the corresponding milestones for ### Deprecated checks #### On the Universal Profile + - **[epar]**: The EPAR table is/was a way of expressing common licensing permissions and restrictions in metadata; almost nothing supported it, and we gave it a chance for quite a while... (issues #4947) - **[kerning_for_non_ligated_sequences]**: "Is there kerning info for non-ligated sequences?" (issue #2954 / https://github.com/simoncozens/fontspector/commit/eaa52447ddc4a42e26b6430841a43026870d8a48) #### On the OpenType Profile diff --git a/Lib/fontbakery/checks/epar.py b/Lib/fontbakery/checks/epar.py deleted file mode 100644 index 04fdc04b32..0000000000 --- a/Lib/fontbakery/checks/epar.py +++ /dev/null @@ -1,25 +0,0 @@ -from fontbakery.prelude import check, Message, INFO - - -@check( - id="epar", - rationale=""" - The EPAR table is/was a way of expressing common licensing permissions and - restrictions in metadata; while almost nothing supported it, Dave Crossland - wonders that having this INFO-level check could help make it more popular. - - More info is available at: - https://davelab6.github.io/epar/ - """, - severity=1, - proposal="https://github.com/fonttools/fontbakery/issues/226", -) -def check_epar(ttFont): - """EPAR table present in font?""" - - if "EPAR" not in ttFont: - yield INFO, Message( - "lacks-EPAR", - "EPAR table not present in font. To learn more see" - " https://github.com/fonttools/fontbakery/issues/818", - ) diff --git a/Lib/fontbakery/profiles/adobefonts.py b/Lib/fontbakery/profiles/adobefonts.py index 49ced3b1ca..fc64da11e7 100644 --- a/Lib/fontbakery/profiles/adobefonts.py +++ b/Lib/fontbakery/profiles/adobefonts.py @@ -50,7 +50,6 @@ "color_cpal_brightness", "contour_count", # was temporarily removed "control_chars", - "epar", "empty_glyph_on_gid1_for_colrv0", "file_size", "fontdata_namecheck", diff --git a/Lib/fontbakery/profiles/fontbureau.py b/Lib/fontbakery/profiles/fontbureau.py index 485c465954..ac0448c726 100644 --- a/Lib/fontbakery/profiles/fontbureau.py +++ b/Lib/fontbakery/profiles/fontbureau.py @@ -11,7 +11,6 @@ "control_chars", "empty_glyph_on_gid1_for_colrv0", "empty_letters", - "epar", "file_size", "fvar_name_entries", "fontdata_namecheck", diff --git a/Lib/fontbakery/profiles/fontwerk.py b/Lib/fontbakery/profiles/fontwerk.py index f4658b9129..6d0f7628c1 100644 --- a/Lib/fontbakery/profiles/fontwerk.py +++ b/Lib/fontbakery/profiles/fontwerk.py @@ -17,7 +17,6 @@ "fontdata_namecheck", ], "pending_review": [ - "epar", "base_has_width", "googlefonts/axes_match", "overlapping_path_segments", diff --git a/Lib/fontbakery/profiles/microsoft.py b/Lib/fontbakery/profiles/microsoft.py index f75ea262c7..9d7bdab1cb 100644 --- a/Lib/fontbakery/profiles/microsoft.py +++ b/Lib/fontbakery/profiles/microsoft.py @@ -23,7 +23,6 @@ "designspace_has_sources", "empty_glyph_on_gid1_for_colrv0", "empty_letters", - "epar", "file_size", "fvar_name_entries", "fontdata_namecheck", diff --git a/Lib/fontbakery/profiles/typenetwork.py b/Lib/fontbakery/profiles/typenetwork.py index d832eb1684..2219569750 100644 --- a/Lib/fontbakery/profiles/typenetwork.py +++ b/Lib/fontbakery/profiles/typenetwork.py @@ -39,7 +39,6 @@ "varfont/unsupported_axes", # Despite discouraging its use, TN accepts fonts with ital axis. ], "pending_review": [ - "epar", "base_has_width", ], "sections": { diff --git a/Lib/fontbakery/profiles/universal.py b/Lib/fontbakery/profiles/universal.py index d4c03fd900..3432b76357 100644 --- a/Lib/fontbakery/profiles/universal.py +++ b/Lib/fontbakery/profiles/universal.py @@ -35,7 +35,6 @@ "control_chars", "empty_glyph_on_gid1_for_colrv0", "empty_letters", - "epar", "family/single_directory", "family/vertical_metrics", "family/win_ascent_and_descent", diff --git a/tests/test_checks_epar.py b/tests/test_checks_epar.py deleted file mode 100644 index 95418b9b6b..0000000000 --- a/tests/test_checks_epar.py +++ /dev/null @@ -1,27 +0,0 @@ -from fontTools.ttLib import TTFont - -from conftest import check_id -from fontbakery.codetesting import ( - assert_PASS, - assert_results_contain, - TEST_FILE, -) -from fontbakery.status import INFO - - -@check_id("epar") -def test_check_epar(check): - """EPAR table present in font?""" - - # Our reference Mada Regular lacks an EPAR table: - ttFont = TTFont(TEST_FILE("mada/Mada-Regular.ttf")) - - # So it must emit an INFO message inviting the designers - # to learn more about it: - assert_results_contain( - check(ttFont), INFO, "lacks-EPAR", "with a font lacking an EPAR table..." - ) - - # add a fake EPAR table to validate the PASS code-path: - ttFont["EPAR"] = "foo" - assert_PASS(check(ttFont), "with a good font...")