Run Fontbakery tests at build time

[sladen]

Fontbakery is a set of tests used for validation before on-boarding new releases into Google Fonts.

Ubuntu needs to pass this for normal releases.

Therefore these need to be integrated as validation at build/post-build time.

[madig]

In a quick test run, fontbakery complained about the presence of the kern table 😁

[moyogo]

Browsers don't use the kern table since there is GPOS kerning. However some applications like PowerPoint only use the kern table. So for web fonts it doesn't make sense to have both kern table and GPOS kerning.

…

On Sat, 16 Dec 2017, 14:57 Nikolaus Waxweiler, ***@***.***> wrote: In a quick test run, fontbakery complained about the presence of the kern table 😁 — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#36 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AB1ao9HdMSNVxoQ5ha-Zlac9Kqp3nupFks5tA9psgaJpZM4RBUBZ> .

[madig]

Okay :) Not sure if one can upload fonts with that "error" then (the current fonts on GF have it anyway). Ideally we only have one binary build for everything?

Here's the fontbakery output for R: Validationrun-R.txt. Interestingly, FontVal seems to have checks for hdmx, VDMX and LTSH. If these tables turn out to be necessary, I can maybe salvage the code.

[moyogo]

I'd say fontbakery needs to allow the kern table, exceptionally is OK.

[moyogo]

Or we produce two sets of fonts.

[anthrotype]

Two sets of fonts is better, the kern table is just dead weight for a web font

[davelab6]

Google Fonts ingests fonts made for download and use as desktops, and does all web font optimizations in the serving pipeline (using libre libraries that anyone can replicate.)

[madig]

Oh, that's neat. So why does it complain about e.g. the kern table in fontbakery, or put differently, does the error matter if the optimization is done anyway?

[davelab6]

TTFs intended for desktop users (and thus the ones intended for GF) should have both KERN and GPOS tables.

[madig]

But fontbakery check-googlefonts ... complains about the kern table.

[davelab6]

Its a bug.

[felipesanches]

It's a fixed bug ;-)

Thanks @sladen for having reported it!

[felipesanches]

felipesanches/fontbakery@ec36a1b