google/fonts

Font Validation report

thlinard opened this issue · 14 comments

I tested all the fonts (minus the duplicate, i.e. Alef, EkMukta, JosefinSansStd, MrBedfort, MrsSaintDelafield, Siemreap, and TerminalDosis) from the GF repo in the Font Book application (macOS 10.12.6). Here the Font Validation report:

Errors (“14 serious errors were found. Do not use these fonts”)
Artifika-Regular.ttf

  • ‘post’ table usability

Aubrey-Regular.ttf

  • ‘post’ table usability

Brawler-Regular.ttf

  • ‘post’ table usability

Molengo-Regular.ttf

  • System Validation

Sarabun-Bold.ttf

  • ‘post’ table usability

Sarabun-BoldItalic.ttf

  • ‘post’ table usability

Sarabun-Italic.ttf

  • ‘post’ table usability

Simonetta-Black.ttf

  • ‘post’ table usability

Simonetta-BlackItalic.ttf

  • ‘post’ table usability

Simonetta-Italic.ttf

  • ‘post’ table usability

Simonetta-Regular.ttf

  • ‘post’ table usability

WireOne.ttf

  • ‘post’ table usability

Warnings (“32 minor problems were found. Proceed with caution”)
Astloch-Bold.ttf

  • ‘name’ table structure

BM-HANNA.ttf

  • ‘name’ table structure

CaesarDressing-Regular.ttf

  • ‘name’ table structure

Flavors-Regular.ttf

  • ‘name’ table structure

FrederickatheGreat-Regular.ttf

  • ‘name’ table structure

Kantumruy-Bold.ttf

  • ‘name’ table structure

Kantumruy-Light.ttf

  • ‘name’ table structure

Kantumruy-Regular.ttf

  • ‘name’ table structure

LeagueScript.ttf

  • ‘name’ table structure

Padauk-Bold.ttf

  • ‘name’ table structure

Padauk-Regular.ttf

  • ‘name’ table structure

SeoulHangang-Light.ttf

  • ‘sfnt’ required tables
  • ‘vmtx’ table usability

SeoulHangang-Medium.ttf

  • ‘sfnt’ required tables
  • ‘vmtx’ table usability

SeoulHangangCondensed-Bold.ttf

  • ‘kern’ table structure and contents

SeoulHangangCondensed-BoldL.ttf

  • ‘kern’ table structure and contents

SeoulHangangCondensed-ExtraBold.ttf

  • ‘kern’ table structure and contents

SeoulHangangCondensed-Light.ttf

  • ‘kern’ table structure and contents

SeoulHangangCondensed-Medium.ttf

  • ‘kern’ table structure and contents

SeoulNamsan-Bold.ttf

  • ‘sfnt’ required tables
  • ‘vmtx’ table usability

SeoulNamsan-Light.ttf

  • ‘sfnt’ required tables
  • ‘vmtx’ table usability

SeoulNamsan-Medium.ttf

  • ‘sfnt’ required tables
  • ‘vmtx’ table usability

SeoulNamsanCondensed-Black.ttf

  • ‘kern’ table structure and contents

SeoulNamsanCondensed-Bold.ttf

  • ‘kern’ table structure and contents

SeoulNamsanCondensed-ExtraBold.ttf

  • ‘kern’ table structure and contents

SeoulNamsanCondensed-Light.ttf

  • ‘kern’ table structure and contents

SeoulNamsanCondensed-Medium.ttf

  • ‘kern’ table structure and contents

StardosStencil-Regular.ttf

  • ‘fpgm’ table instructions

I didn't checked the mentioned issues further.

Fonts install and validate fine on OS 10.11.6. I think rebuilding sources in Glyphs App will resolve the issue.

Artifika fixed in cyrealtype/Artifika@4ae1930 Thanks Alexei!

SeoulHangang-Light.ttf
SeoulHangang-Medium.ttf
fixed in #1333

SeoulNamsan-Bold.ttf
SeoulNamsan-Light.ttf
SeoulNamsan-Medium.ttf
fixed in #1334

Thanks @laerm0!

Aubrey fixed in cyrealtype/Aubrey@a2d42ee Thanks @alexeiva!

Wire One (in fact, simply Wire now) fixed in cyrealtype/Wire-One@1e29b98 Thanks @alexeiva!

It should stay Wire One, I fixed that. Thanks for spotting

Brawler fixed here https://github.com/cyrealtype/Brawler/releases/tag/v1.100
cyrealtype/Brawler@b3737a7

Thanks @thlinard, I am going to work on this list as well. I'll start from the bottom so hopefully we don't do the same thing. :)

Oh, I only maintain the list up-to-date. You do the hard work. 😄
But if you don't want to do the same thing, it might be better to get in touch with Alexei, if he still has plans for updates.

Ah, ha, okay. 😆 Thanks!

@alexeiva I'll just skip any Cyreal fonts on this list for now, okay?

@laerm0 that's fine, yes

Okay, looks like I've gone through them all. Thanks for starting the ball rolling, @thlinard.

Thank you very much!

https://twitter.com/paulbunyar/status/966334413809184768 reported an issue with Wire One post table. I'll try to get these open PRs flushed through soon

Update (@m4rc1e can you reopen?): all the fonts tested again with Font Book from macOS 11.5. Apart from a new error (Handjet), the others had already been fixed but some never made it into production.

Errors (“Serious errors were found. Do not use these fonts”)

Handjet (PR #2679) : bad glyf table (MS Font Validator, after ~30 minutes: "Duplicated contours: Glyph index 1599. Unable to perform test due to previously detected errors: Glyph index 1599 Test: ValidateSimpContMisor")

Artifika: post table usability fails (fixed upstream, but never had a PR here)
Aubrey: post table usability fails (fixed upstream, but never had a PR here)
Brawler: post table usability fails (fixed upstream, but never had a PR here)
Wire One: post table usability fails (fixed upstream, but never had a PR here)

Warnings (“Minor problems were found. Proceed with caution”)

BM-Hanna: name table structure (PR #1364 and #1709 closed)

Caesar Dressing: name table structure (MS Font Validator: "The Mac PostScript string does not match the Microsoft PostScript string" — PR #1363 closed)

SeoulHangang L + M
SeoulNamsan N + L + M (Early acces)
Required sfnt tables
vmtx table usability fails (MS Font Validator: "GSUB: The table is invalid. The directory entry length is zero; vmtx: The table length is incorrect" — PR #1333 and #1334 closed)

SeoulHangang CB + CEB + CL
SeoulNamsan CBL + CEB + CL + CM (Early acces)
structure and content of the kern table (MS Font Validator: "kern: The subtable's rangeShift value does not match the calculated value. subtable #0, calc = 36, actual = 61732" — PR #1333 and #1334 closed)