Skip to content

Commit 83966c8

Browse files
committed
Deprecate opentype/dsig
Removed from the 'OpenType' profile. Merged into 'unwanted_tables' on the 'Universal' profile. (issue #4865)
1 parent 994ff35 commit 83966c8

File tree

8 files changed

+10
-71
lines changed

8 files changed

+10
-71
lines changed

CHANGELOG.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,9 @@ Below are the noteworthy changes from each release.
22
A more detailed list of changes is available in the corresponding milestones for each release in the Github issue tracker (https://github.com/googlefonts/fontbakery/milestones?state=closed).
33

44
## Upcoming release: 0.13.0 (a4?) (2024-Nov-??)
5-
- ...
5+
### Deprecated checks
6+
#### Removed from the OpenType profile
7+
- **DEPRECATED - [opentype/dsig]:** Merged into **[unwanted_tables]** on the `Universal` profile. (issue #4865)
68

79

810
## 0.13.0a3 (2024-Oct-25)

Lib/fontbakery/checks/opentype/dsig.py

Lines changed: 0 additions & 38 deletions
This file was deleted.

Lib/fontbakery/checks/tables.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -128,6 +128,12 @@ def check_required_tables(ttFont, config, is_variable_font):
128128
def check_unwanted_tables(ttFont):
129129
"""Are there unwanted tables?"""
130130
UNWANTED_TABLES = {
131+
"DSIG": (
132+
"This font has a digital signature (DSIG table) which is only required"
133+
" - even if only a placeholder - on old programs like MS Office 2013"
134+
" in order to work properly.\n"
135+
"The current recommendation is to completely remove the DSIG table."
136+
),
131137
"FFTM": "Table contains redundant FontForge timestamp info",
132138
"TTFA": "Redundant TTFAutohint table",
133139
"TSI0": "Table contains data only used in VTT",

Lib/fontbakery/profiles/adobefonts.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@
2828
"notofonts/unicode_range_bits", # so maybe there's still some change they may be considered useful here?
2929
#
3030
"opentype/caret_slope",
31-
"opentype/dsig",
3231
"opentype/fsselection",
3332
"opentype/gdef_mark_chars",
3433
"opentype/gdef_non_mark_chars",

Lib/fontbakery/profiles/opentype.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
"opentype/cff_call_depth",
99
"opentype/cff_deprecated_operators",
1010
"opentype/code_pages",
11-
"opentype/dsig",
1211
"opentype/family/bold_italic_unique_for_nameid1",
1312
"opentype/family/consistent_family_name",
1413
"opentype/family/equal_font_versions",

docs/source/fontbakery/checks/opentype.rst

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,6 @@ CFF table checks
1414
.. automodule:: fontbakery.checks.opentype.cff
1515
:members:
1616

17-
DSIG table checks
18-
*****************
19-
20-
.. automodule:: fontbakery.checks.opentype.dsig
21-
:members:
22-
2317
fvar table checks
2418
*****************
2519

tests/test_checks_opentype_dsig.py

Lines changed: 0 additions & 24 deletions
This file was deleted.

tests/test_checks_universal.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -684,6 +684,7 @@ def test_check_unwanted_tables():
684684
check = CheckTester("unwanted_tables")
685685

686686
unwanted_tables = [
687+
"DSIG",
687688
"FFTM", # FontForge
688689
"TTFA", # TTFAutohint
689690
"TSI0", # TSI* = VTT

0 commit comments

Comments
 (0)