Skip to content

Cairo: Version 3.130;gftools[0.9.24.dev9+gaf4c611] added#5924

Closed
yanone wants to merge 5 commits intomainfrom
gftools_packager_ofl_cairo
Closed

Cairo: Version 3.130;gftools[0.9.24.dev9+gaf4c611] added#5924
yanone wants to merge 5 commits intomainfrom
gftools_packager_ofl_cairo

Conversation

@yanone
Copy link
Copy Markdown
Contributor

@yanone yanone commented Feb 23, 2023

444238f: [gftools-packager] Cairo: Version 3.130;gftools[0.9.24.dev9+gaf4c611] added

… added

* Cairo Version 3.130;gftools[0.9.24.dev9+gaf4c611] taken from the upstream repo https://github.com/Gue3bara/Cairo at commit Gue3bara/Cairo@617ed31.
@yanone yanone linked an issue Feb 23, 2023 that may be closed by this pull request
@yanone yanone added - Ready for Review III Improve rendering / layout by addition of manual hinting, kerning or else I Font Bug labels Feb 23, 2023
@github-actions
Copy link
Copy Markdown
Contributor

Fontbakery report

Fontbakery version: 0.8.10

[1] Family checks
INFO: Check axis ordering on the STAT table. (com.google.fonts/check/STAT/axis_order)
  • INFO From a total of 1 font files, 0 of them (0.00%) lack a STAT table.

    And these are the most common STAT axis orderings:
    ('slnt-wght', 1) [code: summary]


[21] Cairo[slnt,wght].ttf
🔥 FAIL: Does DESCRIPTION file contain broken links? (com.google.fonts/check/description/broken_links)
  • 🔥 FAIL The following links are broken in the DESCRIPTION file:
    https://gaber.design/ (status code: 404) [code: broken-links]
🔥 FAIL: Check METADATA.pb parse correctly. (com.google.fonts/check/metadata/parses)
  • 🔥 FAIL Family metadata at ofl/cairo failed to parse.
    TRACEBACK:
    34:1 : Message type "google.fonts_public.FamilyProto" has no field named "primary_script". [code: parsing-error]
🔥 FAIL: Check Google Fonts glyph coverage. (com.google.fonts/check/glyph_coverage)
  • 🔥 FAIL Missing required codepoints:

    • 0x0308 (COMBINING DIAERESIS)

    • 0x0300 (COMBINING GRAVE ACCENT)

    • 0x0301 (COMBINING ACUTE ACCENT)

    • 0x030B (COMBINING DOUBLE ACUTE ACCENT)

    • 0x0304 (COMBINING MACRON)

    • 0x0132 (LATIN CAPITAL LIGATURE IJ)

    • 0x1E9E (LATIN CAPITAL LETTER SHARP S)

    • 0x0133 (LATIN SMALL LIGATURE IJ)

    • 0x0307 (COMBINING DOT ABOVE)

    • 0x0302 (COMBINING CIRCUMFLEX ACCENT)

    • And 7 more.

Use -F or --full-lists to disable shortening of long lists. [code: missing-codepoints]

🔥 FAIL: Check variable font instances (com.google.fonts/check/fvar_instances)
  • 🔥 FAIL fvar instances are incorrect:
  • Add missing instances
Name current expected
ExtraLight Italic N/A wght=200.0, slnt=-11.0
ExtraLight wght=200.0, slnt=0.0 wght=200.0, slnt=0.0
Light Italic N/A wght=300.0, slnt=-11.0
Light wght=300.0, slnt=0.0 wght=300.0, slnt=0.0
Italic N/A wght=400.0, slnt=-11.0
Regular wght=400.0, slnt=0.0 wght=400.0, slnt=0.0
Medium Italic N/A wght=500.0, slnt=-11.0
Medium wght=500.0, slnt=0.0 wght=500.0, slnt=0.0
SemiBold Italic N/A wght=600.0, slnt=-11.0
SemiBold wght=600.0, slnt=0.0 wght=600.0, slnt=0.0
Bold Italic N/A wght=700.0, slnt=-11.0
Bold wght=700.0, slnt=0.0 wght=700.0, slnt=0.0
ExtraBold Italic N/A wght=800.0, slnt=-11.0
ExtraBold wght=800.0, slnt=0.0 wght=800.0, slnt=0.0
Black Italic N/A wght=900.0, slnt=-11.0
Black wght=900.0, slnt=0.0 wght=900.0, slnt=0.0
WARN: Check copyright namerecords match license file. (com.google.fonts/check/name/license)
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN For now we're still accepting http URLs, but you should consider using https instead.
    [code: http]
WARN: License URL matches License text on name table? (com.google.fonts/check/name/license_url)
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=14] [code: http-in-license-info]
  • WARN For now we're still accepting http URLs, but you should consider using https instead.
    [code: http]
WARN: A static fonts directory with at least two fonts must accompany variable fonts (com.google.fonts/check/repo/vf_has_static_fonts)
  • WARN Please consider adding a subdirectory called "static/" and including in it static font files. [code: missing]
WARN: Ensure fonts have ScriptLangTags declared on the 'meta' table. (com.google.fonts/check/meta/script_lang_tags)
  • WARN This font file does not have a 'meta' table. [code: lacks-meta-table]
WARN: Glyph names are all valid? (com.google.fonts/check/valid_glyphnames)
  • WARN The following glyph names may be too long for some legacy systems which may expect a maximum 31-char length limit:
    alefMaksura_alefMaksuraar.fina.alt, yehHamzaabove_yehHamzaabovear.fina, alefMaksura_noonar.fina.alt.color0, alefMaksura_noonar.fina.alt.color1, noon_yehHamzaabovear.fina.color0, noon_yehHamzaabovear.fina.color1, noon_yehHamzaabovear.fina.color2, seen_yehHamzaabovear.fina.color0, seen_yehHamzaabovear.fina.color1, sheen_yehHamzaabovear.fina.color0 and 13 more.

Use -F or --full-lists to disable shortening of long lists. [code: legacy-long-names]

WARN: Check font contains no unreachable glyphs (com.google.fonts/check/unreachable_glyphs)
  • WARN The following glyphs could not be reached by codepoint or substitution rules:

    • .notdef.color0

    • AEacute.color0

    • AEacute.color1

    • Aacute.color0

    • Aacute.color1

    • Abreve.color0

    • Abreve.color1

    • Acircumflex.color0

    • Acircumflex.color1

    • Adieresis.color0

    • And 1122 more.

Use -F or --full-lists to disable shortening of long lists.
[code: unreachable-glyphs]

WARN: Ensure dotted circle glyph is present and can attach marks. (com.google.fonts/check/dotted_circle)
  • WARN No dotted circle glyph present [code: missing-dotted-circle]
WARN: Check GDEF mark glyph class doesn't have characters that are not marks. (com.google.fonts/check/gdef_non_mark_chars)
  • WARN The following non-mark characters should not be in the GDEF mark glyph class:
    U+FBB2, U+FBB3, U+FBB4, U+FBB5, U+FBB6, U+FBB7, U+FBB8, U+FBB9, U+FBBD and U+FBBE [code: non-mark-chars]
INFO: Does DESCRIPTION file contain a upstream Git repo URL? (com.google.fonts/check/description/git_url)
INFO: Show hinting filesize impact. (com.google.fonts/check/hinting_impact)
  • INFO Hinting filesize impact:
ofl/cairo/Cairo[slnt,wght].ttf
Dehinted Size 590.3kb
Hinted Size 590.3kb
Increase 24 bytes
Change 0.0 %
[code: size-impact]
INFO: Font has old ttfautohint applied? (com.google.fonts/check/old_ttfautohint)
  • INFO Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 3.130;gftools[0.9.24.dev9+gaf4c611]'] [code: version-not-detected]
INFO: EPAR table present in font? (com.google.fonts/check/epar)
INFO: Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? (com.google.fonts/check/gasp)
  • INFO These are the ppm ranges declared on the gasp table:

PPM <= 65535:
flag = 0x0F
- Use grid-fitting
- Use grayscale rendering
- Use gridfitting with ClearType symmetric smoothing
- Use smoothing along multiple axes with ClearType®
[code: ranges]

INFO: Familyname must be unique according to namecheck.fontdata.com (com.google.fonts/check/fontdata_namecheck)
INFO: Check for font-v versioning. (com.google.fonts/check/fontv)
  • INFO Version string is: "Version 3.130;gftools[0.9.24.dev9+gaf4c611]"
    The version string must ideally include a git commit hash and either a "dev" or a "release" suffix such as in the example below:
    "Version 1.3; git-0d08353-release" [code: bad-format]
INFO: Font contains all required tables? (com.google.fonts/check/required_tables)
  • INFO This font contains the following optional tables:

    • loca

    • prep

    • GPOS

    • GSUB

    • And gasp [code: optional-tables]

INFO: List all superfamily filepaths (com.google.fonts/check/superfamily/list)
  • INFO ofl/cairo [code: family-path]

### Summary
💔 ERROR 🔥 FAIL ⚠ WARN 💤 SKIP ℹ INFO 🍞 PASS 🔎 DEBUG
0 4 8 89 10 116 0
0% 2% 4% 39% 4% 51% 0%

Note: The following loglevels were omitted in this report:

  • SKIP
  • PASS
  • DEBUG

Copy link
Copy Markdown
Collaborator

@m4rc1e m4rc1e left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems some kerns have gone missing for the reh forms

Screenshot 2023-02-23 at 11 31 37

I don't mind pushing this if we've fixed the terrible issues in #5913

@yanone
Copy link
Copy Markdown
Contributor Author

yanone commented Feb 23, 2023

I'll take a look soon. Thanks

@yanone
Copy link
Copy Markdown
Contributor Author

yanone commented Feb 23, 2023

We've actually lost all kerning because of the ongoing RTL kerning issue. I asked Simon to make his dumbifier available for pip to install.

@anthrotype
Copy link
Copy Markdown
Member

dumbifier

what's that?

@yanone
Copy link
Copy Markdown
Contributor Author

yanone commented Feb 23, 2023

Simon's solution to the impossibility of glyphsLib to support what Glyphs itself supports, "dumbing" down all smart stuff in Glyphs files without caring about round-tripping: https://github.com/simoncozens/dumbifier

It's a glyphsLib-based pre-processor. He solved the RTL-kerning in 13 lines of code: https://github.com/simoncozens/dumbifier/blob/main/Lib/dumbifier/rtl_kerning.py

I don't know when Simon was going to make that public. He did discuss it with the onboarding team. cc @simoncozens

@anthrotype
Copy link
Copy Markdown
Member

interesting. Can we please have those 13 lines of code in glyphslib and be done with this? :)

@anthrotype
Copy link
Copy Markdown
Member

@yanone
Copy link
Copy Markdown
Contributor Author

yanone commented Feb 23, 2023

Please discuss this with Simon. Note that there are other processors in there (smart components, (or was it corner components?) other sanitizers), and I proposed one more layer sanitizer that he so far didn't incorporate.

Personally I don't care where what code resides. My job is to onboard fonts. What I notice is an ever-growing disconnection between what glyphsLib does and what we need it to do. So if Simon's aim is to host a package where he can quickly add solutions in a few lines of code, I support that 100%. But again, in the end I don't care. Stuff needs to work.

@yanone
Copy link
Copy Markdown
Contributor Author

yanone commented Feb 23, 2023

Another thought would be to include the dumbifier as an external dependency into glyphsLib and have an official pre-processing stage by default. It just needs to be understood by all parties that these dumbifications aren't round-trippable.

If round-tripping is a quasi-requirement of glyphsLib, then having the preprocessor in a separate library makes a lot of sense, tho sadly then it won't be active by default and available only to those who know about its existence.

@anthrotype
Copy link
Copy Markdown
Member

I don't think we need that. We already said we're ok with the RTL kerning not be rountrippable.

@anthrotype
Copy link
Copy Markdown
Member

anthrotype commented Feb 23, 2023

hmm that rtl_kerning dumbifier looks suspiciously small, and in fact the tests @yanone wrote for googlefonts/glyphsLib#838 do not pass when I apply my patch. It looks like the groups aren't updated as expected (in fact I only touch the kerning.py module, not the groups.py).
Mind taking a quick look at googlefonts/glyphsLib#865?

@yanone
Copy link
Copy Markdown
Contributor Author

yanone commented Feb 23, 2023

I'll check it tomorrow.

@yanone
Copy link
Copy Markdown
Contributor Author

yanone commented Feb 24, 2023

@m4rc1e
I committed a new font file with fixed kerning (using my old glyphsLib RTL kerning PR). I reported the red layer issue here but I don't expect a solution to be found anytime soon (and anyway I won't be able to use an improved glyphsLib version while simultaneously having RTL kerning), so IMO please go ahead with the update so that at least the macOS rendering disaster can be fixed before I'm off next week.
The red layers aren't a regression anyway; they're already visible in the live font. Will address that as soon as possible.

@github-actions
Copy link
Copy Markdown
Contributor

Fontbakery report

Fontbakery version: 0.8.10

[1] Family checks
INFO: Check axis ordering on the STAT table. (com.google.fonts/check/STAT/axis_order)
  • INFO From a total of 1 font files, 0 of them (0.00%) lack a STAT table.

    And these are the most common STAT axis orderings:
    ('slnt-wght', 1) [code: summary]


[21] Cairo[slnt,wght].ttf
🔥 FAIL: Does DESCRIPTION file contain broken links? (com.google.fonts/check/description/broken_links)
  • 🔥 FAIL The following links are broken in the DESCRIPTION file:
    https://gaber.design/ (status code: 404) [code: broken-links]
🔥 FAIL: Check METADATA.pb parse correctly. (com.google.fonts/check/metadata/parses)
  • 🔥 FAIL Family metadata at ofl/cairo failed to parse.
    TRACEBACK:
    34:1 : Message type "google.fonts_public.FamilyProto" has no field named "primary_script". [code: parsing-error]
🔥 FAIL: Check Google Fonts glyph coverage. (com.google.fonts/check/glyph_coverage)
  • 🔥 FAIL Missing required codepoints:

    • 0x0308 (COMBINING DIAERESIS)

    • 0x0300 (COMBINING GRAVE ACCENT)

    • 0x0301 (COMBINING ACUTE ACCENT)

    • 0x030B (COMBINING DOUBLE ACUTE ACCENT)

    • 0x0304 (COMBINING MACRON)

    • 0x0132 (LATIN CAPITAL LIGATURE IJ)

    • 0x1E9E (LATIN CAPITAL LETTER SHARP S)

    • 0x0133 (LATIN SMALL LIGATURE IJ)

    • 0x0307 (COMBINING DOT ABOVE)

    • 0x0302 (COMBINING CIRCUMFLEX ACCENT)

    • And 7 more.

Use -F or --full-lists to disable shortening of long lists. [code: missing-codepoints]

🔥 FAIL: Check variable font instances (com.google.fonts/check/fvar_instances)
  • 🔥 FAIL fvar instances are incorrect:
  • Add missing instances
Name current expected
ExtraLight Italic N/A wght=200.0, slnt=-11.0
ExtraLight wght=200.0, slnt=0.0 wght=200.0, slnt=0.0
Light Italic N/A wght=300.0, slnt=-11.0
Light wght=300.0, slnt=0.0 wght=300.0, slnt=0.0
Italic N/A wght=400.0, slnt=-11.0
Regular wght=400.0, slnt=0.0 wght=400.0, slnt=0.0
Medium Italic N/A wght=500.0, slnt=-11.0
Medium wght=500.0, slnt=0.0 wght=500.0, slnt=0.0
SemiBold Italic N/A wght=600.0, slnt=-11.0
SemiBold wght=600.0, slnt=0.0 wght=600.0, slnt=0.0
Bold Italic N/A wght=700.0, slnt=-11.0
Bold wght=700.0, slnt=0.0 wght=700.0, slnt=0.0
ExtraBold Italic N/A wght=800.0, slnt=-11.0
ExtraBold wght=800.0, slnt=0.0 wght=800.0, slnt=0.0
Black Italic N/A wght=900.0, slnt=-11.0
Black wght=900.0, slnt=0.0 wght=900.0, slnt=0.0
WARN: Check copyright namerecords match license file. (com.google.fonts/check/name/license)
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN For now we're still accepting http URLs, but you should consider using https instead.
    [code: http]
WARN: License URL matches License text on name table? (com.google.fonts/check/name/license_url)
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=14] [code: http-in-license-info]
  • WARN For now we're still accepting http URLs, but you should consider using https instead.
    [code: http]
WARN: A static fonts directory with at least two fonts must accompany variable fonts (com.google.fonts/check/repo/vf_has_static_fonts)
  • WARN Please consider adding a subdirectory called "static/" and including in it static font files. [code: missing]
WARN: Ensure fonts have ScriptLangTags declared on the 'meta' table. (com.google.fonts/check/meta/script_lang_tags)
  • WARN This font file does not have a 'meta' table. [code: lacks-meta-table]
WARN: Glyph names are all valid? (com.google.fonts/check/valid_glyphnames)
  • WARN The following glyph names may be too long for some legacy systems which may expect a maximum 31-char length limit:
    alefMaksura_alefMaksuraar.fina.alt, yehHamzaabove_yehHamzaabovear.fina, alefMaksura_noonar.fina.alt.color0, alefMaksura_noonar.fina.alt.color1, noon_yehHamzaabovear.fina.color0, noon_yehHamzaabovear.fina.color1, noon_yehHamzaabovear.fina.color2, seen_yehHamzaabovear.fina.color0, seen_yehHamzaabovear.fina.color1, sheen_yehHamzaabovear.fina.color0 and 13 more.

Use -F or --full-lists to disable shortening of long lists. [code: legacy-long-names]

WARN: Check font contains no unreachable glyphs (com.google.fonts/check/unreachable_glyphs)
  • WARN The following glyphs could not be reached by codepoint or substitution rules:

    • .notdef.color0

    • AEacute.color0

    • AEacute.color1

    • Aacute.color0

    • Aacute.color1

    • Abreve.color0

    • Abreve.color1

    • Acircumflex.color0

    • Acircumflex.color1

    • Adieresis.color0

    • And 1122 more.

Use -F or --full-lists to disable shortening of long lists.
[code: unreachable-glyphs]

WARN: Ensure dotted circle glyph is present and can attach marks. (com.google.fonts/check/dotted_circle)
  • WARN No dotted circle glyph present [code: missing-dotted-circle]
WARN: Check GDEF mark glyph class doesn't have characters that are not marks. (com.google.fonts/check/gdef_non_mark_chars)
  • WARN The following non-mark characters should not be in the GDEF mark glyph class:
    U+FBB2, U+FBB3, U+FBB4, U+FBB5, U+FBB6, U+FBB7, U+FBB8, U+FBB9, U+FBBD and U+FBBE [code: non-mark-chars]
INFO: Does DESCRIPTION file contain a upstream Git repo URL? (com.google.fonts/check/description/git_url)
INFO: Show hinting filesize impact. (com.google.fonts/check/hinting_impact)
  • INFO Hinting filesize impact:
ofl/cairo/Cairo[slnt,wght].ttf
Dehinted Size 590.3kb
Hinted Size 590.3kb
Increase 24 bytes
Change 0.0 %
[code: size-impact]
INFO: Font has old ttfautohint applied? (com.google.fonts/check/old_ttfautohint)
  • INFO Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 3.130;gftools[0.9.24.dev9+gaf4c611]'] [code: version-not-detected]
INFO: EPAR table present in font? (com.google.fonts/check/epar)
INFO: Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? (com.google.fonts/check/gasp)
  • INFO These are the ppm ranges declared on the gasp table:

PPM <= 65535:
flag = 0x0F
- Use grid-fitting
- Use grayscale rendering
- Use gridfitting with ClearType symmetric smoothing
- Use smoothing along multiple axes with ClearType®
[code: ranges]

INFO: Familyname must be unique according to namecheck.fontdata.com (com.google.fonts/check/fontdata_namecheck)
INFO: Check for font-v versioning. (com.google.fonts/check/fontv)
  • INFO Version string is: "Version 3.130;gftools[0.9.24.dev9+gaf4c611]"
    The version string must ideally include a git commit hash and either a "dev" or a "release" suffix such as in the example below:
    "Version 1.3; git-0d08353-release" [code: bad-format]
INFO: Font contains all required tables? (com.google.fonts/check/required_tables)
  • INFO This font contains the following optional tables:

    • loca

    • prep

    • GPOS

    • GSUB

    • And gasp [code: optional-tables]

INFO: List all superfamily filepaths (com.google.fonts/check/superfamily/list)
  • INFO ofl/cairo [code: family-path]

### Summary
💔 ERROR 🔥 FAIL ⚠ WARN 💤 SKIP ℹ INFO 🍞 PASS 🔎 DEBUG
0 4 8 89 10 116 0
0% 2% 4% 39% 4% 51% 0%

Note: The following loglevels were omitted in this report:

  • SKIP
  • PASS
  • DEBUG

@github-actions
Copy link
Copy Markdown
Contributor

Fontbakery report

Fontbakery version: 0.8.10

[1] Family checks
INFO: Check axis ordering on the STAT table. (com.google.fonts/check/STAT/axis_order)
  • INFO From a total of 1 font files, 0 of them (0.00%) lack a STAT table.

    And these are the most common STAT axis orderings:
    ('slnt-wght', 1) [code: summary]


[19] CairoPlay[slnt,wght].ttf
🔥 FAIL: Check METADATA.pb parse correctly. (com.google.fonts/check/metadata/parses)
  • 🔥 FAIL Family metadata at ofl/cairoplay failed to parse.
    TRACEBACK:
    34:1 : Message type "google.fonts_public.FamilyProto" has no field named "primary_script". [code: parsing-error]
🔥 FAIL: Check Google Fonts glyph coverage. (com.google.fonts/check/glyph_coverage)
  • 🔥 FAIL Missing required codepoints:

    • 0x0308 (COMBINING DIAERESIS)

    • 0x0300 (COMBINING GRAVE ACCENT)

    • 0x0301 (COMBINING ACUTE ACCENT)

    • 0x030B (COMBINING DOUBLE ACUTE ACCENT)

    • 0x0304 (COMBINING MACRON)

    • 0x0132 (LATIN CAPITAL LIGATURE IJ)

    • 0x1E9E (LATIN CAPITAL LETTER SHARP S)

    • 0x0133 (LATIN SMALL LIGATURE IJ)

    • 0x0307 (COMBINING DOT ABOVE)

    • 0x0302 (COMBINING CIRCUMFLEX ACCENT)

    • And 7 more.

Use -F or --full-lists to disable shortening of long lists. [code: missing-codepoints]

🔥 FAIL: Check variable font instances (com.google.fonts/check/fvar_instances)
  • 🔥 FAIL fvar instances are incorrect:
  • Add missing instances
Name current expected
ExtraLight Italic N/A wght=200.0, slnt=-11.0
ExtraLight wght=200.0, slnt=0.0 wght=200.0, slnt=0.0
Light Italic N/A wght=300.0, slnt=-11.0
Light wght=300.0, slnt=0.0 wght=300.0, slnt=0.0
Italic N/A wght=400.0, slnt=-11.0
Regular wght=400.0, slnt=0.0 wght=400.0, slnt=0.0
Medium Italic N/A wght=500.0, slnt=-11.0
Medium wght=500.0, slnt=0.0 wght=500.0, slnt=0.0
SemiBold Italic N/A wght=600.0, slnt=-11.0
SemiBold wght=600.0, slnt=0.0 wght=600.0, slnt=0.0
Bold Italic N/A wght=700.0, slnt=-11.0
Bold wght=700.0, slnt=0.0 wght=700.0, slnt=0.0
ExtraBold Italic N/A wght=800.0, slnt=-11.0
ExtraBold wght=800.0, slnt=0.0 wght=800.0, slnt=0.0
Black Italic N/A wght=900.0, slnt=-11.0
Black wght=900.0, slnt=0.0 wght=900.0, slnt=0.0
WARN: Check copyright namerecords match license file. (com.google.fonts/check/name/license)
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN For now we're still accepting http URLs, but you should consider using https instead.
    [code: http]
WARN: License URL matches License text on name table? (com.google.fonts/check/name/license_url)
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=14] [code: http-in-license-info]
  • WARN For now we're still accepting http URLs, but you should consider using https instead.
    [code: http]
WARN: A static fonts directory with at least two fonts must accompany variable fonts (com.google.fonts/check/repo/vf_has_static_fonts)
  • WARN Please consider adding a subdirectory called "static/" and including in it static font files. [code: missing]
WARN: Ensure fonts have ScriptLangTags declared on the 'meta' table. (com.google.fonts/check/meta/script_lang_tags)
  • WARN This font file does not have a 'meta' table. [code: lacks-meta-table]
WARN: Glyph names are all valid? (com.google.fonts/check/valid_glyphnames)
  • WARN The following glyph names may be too long for some legacy systems which may expect a maximum 31-char length limit:
    alefMaksura_alefMaksuraar.fina.alt, yehHamzaabove_yehHamzaabovear.fina, alefMaksura_noonar.fina.alt.color0, alefMaksura_noonar.fina.alt.color1, noon_yehHamzaabovear.fina.color0, noon_yehHamzaabovear.fina.color1, noon_yehHamzaabovear.fina.color2, seen_yehHamzaabovear.fina.color0, seen_yehHamzaabovear.fina.color1, sheen_yehHamzaabovear.fina.color0 and 13 more.

Use -F or --full-lists to disable shortening of long lists. [code: legacy-long-names]

WARN: Check font contains no unreachable glyphs (com.google.fonts/check/unreachable_glyphs)
  • WARN The following glyphs could not be reached by codepoint or substitution rules:

    • at.cap

    • braceleft.cap

    • braceright.cap

    • bracketleft.cap

    • bracketright.cap

    • connection_r

    • dotcenterar

    • eight.pnum

    • emdash.cap

    • endash.cap

    • And 20 more.

Use -F or --full-lists to disable shortening of long lists.
[code: unreachable-glyphs]

WARN: Ensure dotted circle glyph is present and can attach marks. (com.google.fonts/check/dotted_circle)
  • WARN No dotted circle glyph present [code: missing-dotted-circle]
WARN: Check GDEF mark glyph class doesn't have characters that are not marks. (com.google.fonts/check/gdef_non_mark_chars)
  • WARN The following non-mark characters should not be in the GDEF mark glyph class:
    U+FBB2, U+FBB3, U+FBB4, U+FBB5, U+FBB6, U+FBB7, U+FBB8, U+FBB9, U+FBBD and U+FBBE [code: non-mark-chars]
INFO: Does DESCRIPTION file contain a upstream Git repo URL? (com.google.fonts/check/description/git_url)
INFO: Show hinting filesize impact. (com.google.fonts/check/hinting_impact)
  • INFO Hinting filesize impact:
ofl/cairoplay/CairoPlay[slnt,wght].ttf
Dehinted Size 594.1kb
Hinted Size 594.1kb
Increase 24 bytes
Change 0.0 %
[code: size-impact]
INFO: Font has old ttfautohint applied? (com.google.fonts/check/old_ttfautohint)
  • INFO Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 3.130;gftools[0.9.24]'] [code: version-not-detected]
INFO: EPAR table present in font? (com.google.fonts/check/epar)
INFO: Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? (com.google.fonts/check/gasp)
  • INFO These are the ppm ranges declared on the gasp table:

PPM <= 65535:
flag = 0x0F
- Use grid-fitting
- Use grayscale rendering
- Use gridfitting with ClearType symmetric smoothing
- Use smoothing along multiple axes with ClearType®
[code: ranges]

INFO: Check for font-v versioning. (com.google.fonts/check/fontv)
  • INFO Version string is: "Version 3.130;gftools[0.9.24]"
    The version string must ideally include a git commit hash and either a "dev" or a "release" suffix such as in the example below:
    "Version 1.3; git-0d08353-release" [code: bad-format]
INFO: Font contains all required tables? (com.google.fonts/check/required_tables)
  • INFO This font contains the following optional tables:

    • loca

    • prep

    • GPOS

    • GSUB

    • And gasp [code: optional-tables]

INFO: List all superfamily filepaths (com.google.fonts/check/superfamily/list)
  • INFO ofl/cairoplay [code: family-path]

### Summary
💔 ERROR 🔥 FAIL ⚠ WARN 💤 SKIP ℹ INFO 🍞 PASS 🔎 DEBUG
0 3 8 89 9 118 0
0% 1% 4% 39% 4% 52% 0%

Note: The following loglevels were omitted in this report:

  • SKIP
  • PASS
  • DEBUG

@anthrotype
Copy link
Copy Markdown
Member

@yanone I have updated my glyphsLib PR (googlefonts/glyphsLib#865) to swap left/right groups for glyphs used in kerningRTL, do you mind giving it a try with Cairo to see if it works as you expect?

@yanone
Copy link
Copy Markdown
Contributor Author

yanone commented Feb 28, 2023

@anthrotype IT WORKS!

@github-actions
Copy link
Copy Markdown
Contributor

Fontbakery report

Fontbakery version: 0.8.10

[1] Family checks
INFO: Check axis ordering on the STAT table. (com.google.fonts/check/STAT/axis_order)
  • INFO From a total of 1 font files, 0 of them (0.00%) lack a STAT table.

    And these are the most common STAT axis orderings:
    ('slnt-wght', 1) [code: summary]


[21] Cairo[slnt,wght].ttf
🔥 FAIL: Does DESCRIPTION file contain broken links? (com.google.fonts/check/description/broken_links)
  • 🔥 FAIL The following links are broken in the DESCRIPTION file:
    https://gaber.design/ (status code: 404) [code: broken-links]
🔥 FAIL: Check METADATA.pb parse correctly. (com.google.fonts/check/metadata/parses)
  • 🔥 FAIL Family metadata at ofl/cairo failed to parse.
    TRACEBACK:
    34:1 : Message type "google.fonts_public.FamilyProto" has no field named "primary_script". [code: parsing-error]
🔥 FAIL: Check Google Fonts glyph coverage. (com.google.fonts/check/glyph_coverage)
  • 🔥 FAIL Missing required codepoints:

    • 0x0308 (COMBINING DIAERESIS)

    • 0x0300 (COMBINING GRAVE ACCENT)

    • 0x0301 (COMBINING ACUTE ACCENT)

    • 0x030B (COMBINING DOUBLE ACUTE ACCENT)

    • 0x0304 (COMBINING MACRON)

    • 0x0132 (LATIN CAPITAL LIGATURE IJ)

    • 0x1E9E (LATIN CAPITAL LETTER SHARP S)

    • 0x0133 (LATIN SMALL LIGATURE IJ)

    • 0x0307 (COMBINING DOT ABOVE)

    • 0x0302 (COMBINING CIRCUMFLEX ACCENT)

    • And 7 more.

Use -F or --full-lists to disable shortening of long lists. [code: missing-codepoints]

🔥 FAIL: Check variable font instances (com.google.fonts/check/fvar_instances)
  • 🔥 FAIL fvar instances are incorrect:
  • Add missing instances
Name current expected
ExtraLight Italic N/A wght=200.0, slnt=-11.0
ExtraLight wght=200.0, slnt=0.0 wght=200.0, slnt=0.0
Light Italic N/A wght=300.0, slnt=-11.0
Light wght=300.0, slnt=0.0 wght=300.0, slnt=0.0
Italic N/A wght=400.0, slnt=-11.0
Regular wght=400.0, slnt=0.0 wght=400.0, slnt=0.0
Medium Italic N/A wght=500.0, slnt=-11.0
Medium wght=500.0, slnt=0.0 wght=500.0, slnt=0.0
SemiBold Italic N/A wght=600.0, slnt=-11.0
SemiBold wght=600.0, slnt=0.0 wght=600.0, slnt=0.0
Bold Italic N/A wght=700.0, slnt=-11.0
Bold wght=700.0, slnt=0.0 wght=700.0, slnt=0.0
ExtraBold Italic N/A wght=800.0, slnt=-11.0
ExtraBold wght=800.0, slnt=0.0 wght=800.0, slnt=0.0
Black Italic N/A wght=900.0, slnt=-11.0
Black wght=900.0, slnt=0.0 wght=900.0, slnt=0.0
WARN: Check copyright namerecords match license file. (com.google.fonts/check/name/license)
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN For now we're still accepting http URLs, but you should consider using https instead.
    [code: http]
WARN: License URL matches License text on name table? (com.google.fonts/check/name/license_url)
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=14] [code: http-in-license-info]
  • WARN For now we're still accepting http URLs, but you should consider using https instead.
    [code: http]
WARN: A static fonts directory with at least two fonts must accompany variable fonts (com.google.fonts/check/repo/vf_has_static_fonts)
  • WARN Please consider adding a subdirectory called "static/" and including in it static font files. [code: missing]
WARN: Ensure fonts have ScriptLangTags declared on the 'meta' table. (com.google.fonts/check/meta/script_lang_tags)
  • WARN This font file does not have a 'meta' table. [code: lacks-meta-table]
WARN: Glyph names are all valid? (com.google.fonts/check/valid_glyphnames)
  • WARN The following glyph names may be too long for some legacy systems which may expect a maximum 31-char length limit:
    alefMaksura_alefMaksuraar.fina.alt, yehHamzaabove_yehHamzaabovear.fina, alefMaksura_noonar.fina.alt.color0, alefMaksura_noonar.fina.alt.color1, noon_yehHamzaabovear.fina.color0, noon_yehHamzaabovear.fina.color1, noon_yehHamzaabovear.fina.color2, seen_yehHamzaabovear.fina.color0, seen_yehHamzaabovear.fina.color1, sheen_yehHamzaabovear.fina.color0 and 13 more.

Use -F or --full-lists to disable shortening of long lists. [code: legacy-long-names]

WARN: Check font contains no unreachable glyphs (com.google.fonts/check/unreachable_glyphs)
  • WARN The following glyphs could not be reached by codepoint or substitution rules:

    • .notdef.color0

    • AEacute.color0

    • AEacute.color1

    • Aacute.color0

    • Aacute.color1

    • Abreve.color0

    • Abreve.color1

    • Acircumflex.color0

    • Acircumflex.color1

    • Adieresis.color0

    • And 1122 more.

Use -F or --full-lists to disable shortening of long lists.
[code: unreachable-glyphs]

WARN: Ensure dotted circle glyph is present and can attach marks. (com.google.fonts/check/dotted_circle)
  • WARN No dotted circle glyph present [code: missing-dotted-circle]
WARN: Check GDEF mark glyph class doesn't have characters that are not marks. (com.google.fonts/check/gdef_non_mark_chars)
  • WARN The following non-mark characters should not be in the GDEF mark glyph class:
    U+FBB2, U+FBB3, U+FBB4, U+FBB5, U+FBB6, U+FBB7, U+FBB8, U+FBB9, U+FBBD and U+FBBE [code: non-mark-chars]
INFO: Does DESCRIPTION file contain a upstream Git repo URL? (com.google.fonts/check/description/git_url)
INFO: Show hinting filesize impact. (com.google.fonts/check/hinting_impact)
  • INFO Hinting filesize impact:
ofl/cairo/Cairo[slnt,wght].ttf
Dehinted Size 590.3kb
Hinted Size 590.3kb
Increase 24 bytes
Change 0.0 %
[code: size-impact]
INFO: Font has old ttfautohint applied? (com.google.fonts/check/old_ttfautohint)
  • INFO Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 3.130;gftools[0.9.24.dev9+gaf4c611]'] [code: version-not-detected]
INFO: EPAR table present in font? (com.google.fonts/check/epar)
INFO: Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? (com.google.fonts/check/gasp)
  • INFO These are the ppm ranges declared on the gasp table:

PPM <= 65535:
flag = 0x0F
- Use grid-fitting
- Use grayscale rendering
- Use gridfitting with ClearType symmetric smoothing
- Use smoothing along multiple axes with ClearType®
[code: ranges]

INFO: Familyname must be unique according to namecheck.fontdata.com (com.google.fonts/check/fontdata_namecheck)
INFO: Check for font-v versioning. (com.google.fonts/check/fontv)
  • INFO Version string is: "Version 3.130;gftools[0.9.24.dev9+gaf4c611]"
    The version string must ideally include a git commit hash and either a "dev" or a "release" suffix such as in the example below:
    "Version 1.3; git-0d08353-release" [code: bad-format]
INFO: Font contains all required tables? (com.google.fonts/check/required_tables)
  • INFO This font contains the following optional tables:

    • loca

    • prep

    • GPOS

    • GSUB

    • And gasp [code: optional-tables]

INFO: List all superfamily filepaths (com.google.fonts/check/superfamily/list)
  • INFO ofl/cairo [code: family-path]

### Summary
💔 ERROR 🔥 FAIL ⚠ WARN 💤 SKIP ℹ INFO 🍞 PASS 🔎 DEBUG
0 4 8 89 10 116 0
0% 2% 4% 39% 4% 51% 0%

Note: The following loglevels were omitted in this report:

  • SKIP
  • PASS
  • DEBUG

@github-actions
Copy link
Copy Markdown
Contributor

Fontbakery report

Fontbakery version: 0.8.10

[1] Family checks
INFO: Check axis ordering on the STAT table. (com.google.fonts/check/STAT/axis_order)
  • INFO From a total of 1 font files, 0 of them (0.00%) lack a STAT table.

    And these are the most common STAT axis orderings:
    ('slnt-wght', 1) [code: summary]


[19] CairoPlay[slnt,wght].ttf
🔥 FAIL: Check METADATA.pb parse correctly. (com.google.fonts/check/metadata/parses)
  • 🔥 FAIL Family metadata at ofl/cairoplay failed to parse.
    TRACEBACK:
    34:1 : Message type "google.fonts_public.FamilyProto" has no field named "primary_script". [code: parsing-error]
🔥 FAIL: Check Google Fonts glyph coverage. (com.google.fonts/check/glyph_coverage)
  • 🔥 FAIL Missing required codepoints:

    • 0x0308 (COMBINING DIAERESIS)

    • 0x0300 (COMBINING GRAVE ACCENT)

    • 0x0301 (COMBINING ACUTE ACCENT)

    • 0x030B (COMBINING DOUBLE ACUTE ACCENT)

    • 0x0304 (COMBINING MACRON)

    • 0x0132 (LATIN CAPITAL LIGATURE IJ)

    • 0x1E9E (LATIN CAPITAL LETTER SHARP S)

    • 0x0133 (LATIN SMALL LIGATURE IJ)

    • 0x0307 (COMBINING DOT ABOVE)

    • 0x0302 (COMBINING CIRCUMFLEX ACCENT)

    • And 7 more.

Use -F or --full-lists to disable shortening of long lists. [code: missing-codepoints]

🔥 FAIL: Check variable font instances (com.google.fonts/check/fvar_instances)
  • 🔥 FAIL fvar instances are incorrect:
  • Add missing instances
Name current expected
ExtraLight Italic N/A wght=200.0, slnt=-11.0
ExtraLight wght=200.0, slnt=0.0 wght=200.0, slnt=0.0
Light Italic N/A wght=300.0, slnt=-11.0
Light wght=300.0, slnt=0.0 wght=300.0, slnt=0.0
Italic N/A wght=400.0, slnt=-11.0
Regular wght=400.0, slnt=0.0 wght=400.0, slnt=0.0
Medium Italic N/A wght=500.0, slnt=-11.0
Medium wght=500.0, slnt=0.0 wght=500.0, slnt=0.0
SemiBold Italic N/A wght=600.0, slnt=-11.0
SemiBold wght=600.0, slnt=0.0 wght=600.0, slnt=0.0
Bold Italic N/A wght=700.0, slnt=-11.0
Bold wght=700.0, slnt=0.0 wght=700.0, slnt=0.0
ExtraBold Italic N/A wght=800.0, slnt=-11.0
ExtraBold wght=800.0, slnt=0.0 wght=800.0, slnt=0.0
Black Italic N/A wght=900.0, slnt=-11.0
Black wght=900.0, slnt=0.0 wght=900.0, slnt=0.0
WARN: Check copyright namerecords match license file. (com.google.fonts/check/name/license)
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN For now we're still accepting http URLs, but you should consider using https instead.
    [code: http]
WARN: License URL matches License text on name table? (com.google.fonts/check/name/license_url)
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=14] [code: http-in-license-info]
  • WARN For now we're still accepting http URLs, but you should consider using https instead.
    [code: http]
WARN: A static fonts directory with at least two fonts must accompany variable fonts (com.google.fonts/check/repo/vf_has_static_fonts)
  • WARN Please consider adding a subdirectory called "static/" and including in it static font files. [code: missing]
WARN: Ensure fonts have ScriptLangTags declared on the 'meta' table. (com.google.fonts/check/meta/script_lang_tags)
  • WARN This font file does not have a 'meta' table. [code: lacks-meta-table]
WARN: Glyph names are all valid? (com.google.fonts/check/valid_glyphnames)
  • WARN The following glyph names may be too long for some legacy systems which may expect a maximum 31-char length limit:
    alefMaksura_alefMaksuraar.fina.alt, yehHamzaabove_yehHamzaabovear.fina, alefMaksura_noonar.fina.alt.color0, alefMaksura_noonar.fina.alt.color1, noon_yehHamzaabovear.fina.color0, noon_yehHamzaabovear.fina.color1, noon_yehHamzaabovear.fina.color2, seen_yehHamzaabovear.fina.color0, seen_yehHamzaabovear.fina.color1, sheen_yehHamzaabovear.fina.color0 and 13 more.

Use -F or --full-lists to disable shortening of long lists. [code: legacy-long-names]

WARN: Check font contains no unreachable glyphs (com.google.fonts/check/unreachable_glyphs)
  • WARN The following glyphs could not be reached by codepoint or substitution rules:

    • at.cap

    • braceleft.cap

    • braceright.cap

    • bracketleft.cap

    • bracketright.cap

    • connection_r

    • dotcenterar

    • eight.pnum

    • emdash.cap

    • endash.cap

    • And 20 more.

Use -F or --full-lists to disable shortening of long lists.
[code: unreachable-glyphs]

WARN: Ensure dotted circle glyph is present and can attach marks. (com.google.fonts/check/dotted_circle)
  • WARN No dotted circle glyph present [code: missing-dotted-circle]
WARN: Check GDEF mark glyph class doesn't have characters that are not marks. (com.google.fonts/check/gdef_non_mark_chars)
  • WARN The following non-mark characters should not be in the GDEF mark glyph class:
    U+FBB2, U+FBB3, U+FBB4, U+FBB5, U+FBB6, U+FBB7, U+FBB8, U+FBB9, U+FBBD and U+FBBE [code: non-mark-chars]
INFO: Does DESCRIPTION file contain a upstream Git repo URL? (com.google.fonts/check/description/git_url)
INFO: Show hinting filesize impact. (com.google.fonts/check/hinting_impact)
  • INFO Hinting filesize impact:
ofl/cairoplay/CairoPlay[slnt,wght].ttf
Dehinted Size 594.1kb
Hinted Size 594.1kb
Increase 24 bytes
Change 0.0 %
[code: size-impact]
INFO: Font has old ttfautohint applied? (com.google.fonts/check/old_ttfautohint)
  • INFO Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 3.130;gftools[0.9.24]'] [code: version-not-detected]
INFO: EPAR table present in font? (com.google.fonts/check/epar)
INFO: Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? (com.google.fonts/check/gasp)
  • INFO These are the ppm ranges declared on the gasp table:

PPM <= 65535:
flag = 0x0F
- Use grid-fitting
- Use grayscale rendering
- Use gridfitting with ClearType symmetric smoothing
- Use smoothing along multiple axes with ClearType®
[code: ranges]

INFO: Check for font-v versioning. (com.google.fonts/check/fontv)
  • INFO Version string is: "Version 3.130;gftools[0.9.24]"
    The version string must ideally include a git commit hash and either a "dev" or a "release" suffix such as in the example below:
    "Version 1.3; git-0d08353-release" [code: bad-format]
INFO: Font contains all required tables? (com.google.fonts/check/required_tables)
  • INFO This font contains the following optional tables:

    • loca

    • prep

    • GPOS

    • GSUB

    • And gasp [code: optional-tables]

INFO: List all superfamily filepaths (com.google.fonts/check/superfamily/list)
  • INFO ofl/cairoplay [code: family-path]

### Summary
💔 ERROR 🔥 FAIL ⚠ WARN 💤 SKIP ℹ INFO 🍞 PASS 🔎 DEBUG
0 3 8 89 9 118 0
0% 1% 4% 39% 4% 52% 0%

Note: The following loglevels were omitted in this report:

  • SKIP
  • PASS
  • DEBUG

… added

* Cairo Version 3.130;gftools[0.9.24.dev9+gaf4c611] taken from the upstream repo https://github.com/Gue3bara/Cairo at commit Gue3bara/Cairo@617ed31.
@RosaWagner
Copy link
Copy Markdown
Contributor

Updated

Cairo: Version 3.130;gftools[0.9.24.dev9+gaf4c611] added


0521bb6: [gftools-packager] Cairo: Version 3.130;gftools[0.9.24.dev9+gaf4c611] added

@RosaWagner RosaWagner force-pushed the gftools_packager_ofl_cairo branch from 5bfde15 to 0521bb6 Compare March 1, 2023 11:48
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 1, 2023

Fontbakery report

Fontbakery version: 0.8.10

[1] Family checks
INFO: Check axis ordering on the STAT table. (com.google.fonts/check/STAT/axis_order)
  • INFO From a total of 1 font files, 0 of them (0.00%) lack a STAT table.

    And these are the most common STAT axis orderings:
    ('slnt-wght', 1) [code: summary]


[21] Cairo[slnt,wght].ttf
🔥 FAIL: Does DESCRIPTION file contain broken links? (com.google.fonts/check/description/broken_links)
  • 🔥 FAIL The following links are broken in the DESCRIPTION file:
    https://gaber.design/ (status code: 404) [code: broken-links]
🔥 FAIL: Check METADATA.pb parse correctly. (com.google.fonts/check/metadata/parses)
  • 🔥 FAIL Family metadata at ofl/cairo failed to parse.
    TRACEBACK:
    34:1 : Message type "google.fonts_public.FamilyProto" has no field named "primary_script". [code: parsing-error]
🔥 FAIL: Check Google Fonts glyph coverage. (com.google.fonts/check/glyph_coverage)
  • 🔥 FAIL Missing required codepoints:

    • 0x0308 (COMBINING DIAERESIS)

    • 0x0300 (COMBINING GRAVE ACCENT)

    • 0x0301 (COMBINING ACUTE ACCENT)

    • 0x030B (COMBINING DOUBLE ACUTE ACCENT)

    • 0x0304 (COMBINING MACRON)

    • 0x0132 (LATIN CAPITAL LIGATURE IJ)

    • 0x1E9E (LATIN CAPITAL LETTER SHARP S)

    • 0x0133 (LATIN SMALL LIGATURE IJ)

    • 0x0307 (COMBINING DOT ABOVE)

    • 0x0302 (COMBINING CIRCUMFLEX ACCENT)

    • And 7 more.

Use -F or --full-lists to disable shortening of long lists. [code: missing-codepoints]

🔥 FAIL: Check variable font instances (com.google.fonts/check/fvar_instances)
  • 🔥 FAIL fvar instances are incorrect:
  • Add missing instances
Name current expected
ExtraLight Italic N/A wght=200.0, slnt=-11.0
ExtraLight wght=200.0, slnt=0.0 wght=200.0, slnt=0.0
Light Italic N/A wght=300.0, slnt=-11.0
Light wght=300.0, slnt=0.0 wght=300.0, slnt=0.0
Italic N/A wght=400.0, slnt=-11.0
Regular wght=400.0, slnt=0.0 wght=400.0, slnt=0.0
Medium Italic N/A wght=500.0, slnt=-11.0
Medium wght=500.0, slnt=0.0 wght=500.0, slnt=0.0
SemiBold Italic N/A wght=600.0, slnt=-11.0
SemiBold wght=600.0, slnt=0.0 wght=600.0, slnt=0.0
Bold Italic N/A wght=700.0, slnt=-11.0
Bold wght=700.0, slnt=0.0 wght=700.0, slnt=0.0
ExtraBold Italic N/A wght=800.0, slnt=-11.0
ExtraBold wght=800.0, slnt=0.0 wght=800.0, slnt=0.0
Black Italic N/A wght=900.0, slnt=-11.0
Black wght=900.0, slnt=0.0 wght=900.0, slnt=0.0
WARN: Check copyright namerecords match license file. (com.google.fonts/check/name/license)
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN For now we're still accepting http URLs, but you should consider using https instead.
    [code: http]
WARN: License URL matches License text on name table? (com.google.fonts/check/name/license_url)
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=14] [code: http-in-license-info]
  • WARN For now we're still accepting http URLs, but you should consider using https instead.
    [code: http]
WARN: A static fonts directory with at least two fonts must accompany variable fonts (com.google.fonts/check/repo/vf_has_static_fonts)
  • WARN Please consider adding a subdirectory called "static/" and including in it static font files. [code: missing]
WARN: Ensure fonts have ScriptLangTags declared on the 'meta' table. (com.google.fonts/check/meta/script_lang_tags)
  • WARN This font file does not have a 'meta' table. [code: lacks-meta-table]
WARN: Glyph names are all valid? (com.google.fonts/check/valid_glyphnames)
  • WARN The following glyph names may be too long for some legacy systems which may expect a maximum 31-char length limit:
    alefMaksura_alefMaksuraar.fina.alt, yehHamzaabove_yehHamzaabovear.fina, alefMaksura_noonar.fina.alt.color0, alefMaksura_noonar.fina.alt.color1, noon_yehHamzaabovear.fina.color0, noon_yehHamzaabovear.fina.color1, noon_yehHamzaabovear.fina.color2, seen_yehHamzaabovear.fina.color0, seen_yehHamzaabovear.fina.color1, sheen_yehHamzaabovear.fina.color0 and 13 more.

Use -F or --full-lists to disable shortening of long lists. [code: legacy-long-names]

WARN: Check font contains no unreachable glyphs (com.google.fonts/check/unreachable_glyphs)
  • WARN The following glyphs could not be reached by codepoint or substitution rules:

    • .notdef.color0

    • AEacute.color0

    • AEacute.color1

    • Aacute.color0

    • Aacute.color1

    • Abreve.color0

    • Abreve.color1

    • Acircumflex.color0

    • Acircumflex.color1

    • Adieresis.color0

    • And 1122 more.

Use -F or --full-lists to disable shortening of long lists.
[code: unreachable-glyphs]

WARN: Ensure dotted circle glyph is present and can attach marks. (com.google.fonts/check/dotted_circle)
  • WARN No dotted circle glyph present [code: missing-dotted-circle]
WARN: Check GDEF mark glyph class doesn't have characters that are not marks. (com.google.fonts/check/gdef_non_mark_chars)
  • WARN The following non-mark characters should not be in the GDEF mark glyph class:
    U+FBB2, U+FBB3, U+FBB4, U+FBB5, U+FBB6, U+FBB7, U+FBB8, U+FBB9, U+FBBD and U+FBBE [code: non-mark-chars]
INFO: Does DESCRIPTION file contain a upstream Git repo URL? (com.google.fonts/check/description/git_url)
INFO: Show hinting filesize impact. (com.google.fonts/check/hinting_impact)
  • INFO Hinting filesize impact:
ofl/cairo/Cairo[slnt,wght].ttf
Dehinted Size 590.3kb
Hinted Size 590.3kb
Increase 24 bytes
Change 0.0 %
[code: size-impact]
INFO: Font has old ttfautohint applied? (com.google.fonts/check/old_ttfautohint)
  • INFO Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 3.130;gftools[0.9.24.dev9+gaf4c611]'] [code: version-not-detected]
INFO: EPAR table present in font? (com.google.fonts/check/epar)
INFO: Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? (com.google.fonts/check/gasp)
  • INFO These are the ppm ranges declared on the gasp table:

PPM <= 65535:
flag = 0x0F
- Use grid-fitting
- Use grayscale rendering
- Use gridfitting with ClearType symmetric smoothing
- Use smoothing along multiple axes with ClearType®
[code: ranges]

INFO: Familyname must be unique according to namecheck.fontdata.com (com.google.fonts/check/fontdata_namecheck)
INFO: Check for font-v versioning. (com.google.fonts/check/fontv)
  • INFO Version string is: "Version 3.130;gftools[0.9.24.dev9+gaf4c611]"
    The version string must ideally include a git commit hash and either a "dev" or a "release" suffix such as in the example below:
    "Version 1.3; git-0d08353-release" [code: bad-format]
INFO: Font contains all required tables? (com.google.fonts/check/required_tables)
  • INFO This font contains the following optional tables:

    • loca

    • prep

    • GPOS

    • GSUB

    • And gasp [code: optional-tables]

INFO: List all superfamily filepaths (com.google.fonts/check/superfamily/list)
  • INFO ofl/cairo [code: family-path]

### Summary
💔 ERROR 🔥 FAIL ⚠ WARN 💤 SKIP ℹ INFO 🍞 PASS 🔎 DEBUG
0 4 8 89 10 116 0
0% 2% 4% 39% 4% 51% 0%

Note: The following loglevels were omitted in this report:

  • SKIP
  • PASS
  • DEBUG

@yanone
Copy link
Copy Markdown
Contributor Author

yanone commented Mar 6, 2023

Holding this back because Khaled may have fixed the glyphsLib issue that led to the duplicate red layer. Will check today and report back

@m4rc1e
Copy link
Copy Markdown
Collaborator

m4rc1e commented Mar 6, 2023

Perfect. Thanks for the update.

@yanone
Copy link
Copy Markdown
Contributor Author

yanone commented Mar 6, 2023

This was actually the wrong PR. I didn't realize someone made a separate PR for Cairo non-Play. But the kerning issue applies to both, so I'll check

@yanone
Copy link
Copy Markdown
Contributor Author

yanone commented Mar 6, 2023

@m4rc1e Please review again. The red layer issue is resolved, and kerning should work.
This PR now contains both Cairo & Cairo Play updates because of a mistake of mine.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 6, 2023

Fontbakery report

Fontbakery version: 0.8.11

[1] Family checks
INFO: Check axis ordering on the STAT table. (com.google.fonts/check/STAT/axis_order)
  • INFO From a total of 1 font files, 0 of them (0.00%) lack a STAT table.

    And these are the most common STAT axis orderings:
    ('slnt-wght', 1) [code: summary]


[23] Cairo[slnt,wght].ttf
🔥 FAIL: Does DESCRIPTION file contain broken links? (com.google.fonts/check/description/broken_links)
  • 🔥 FAIL The following links are broken in the DESCRIPTION file:
    https://gaber.design/ (status code: 404) [code: broken-links]
🔥 FAIL: Check Google Fonts glyph coverage. (com.google.fonts/check/glyph_coverage)
  • 🔥 FAIL Missing required codepoints:

    • 0x0308 (COMBINING DIAERESIS)

    • 0x0300 (COMBINING GRAVE ACCENT)

    • 0x0301 (COMBINING ACUTE ACCENT)

    • 0x030B (COMBINING DOUBLE ACUTE ACCENT)

    • 0x0304 (COMBINING MACRON)

    • 0x0132 (LATIN CAPITAL LIGATURE IJ)

    • 0x1E9E (LATIN CAPITAL LETTER SHARP S)

    • 0x0133 (LATIN SMALL LIGATURE IJ)

    • 0x0307 (COMBINING DOT ABOVE)

    • 0x0302 (COMBINING CIRCUMFLEX ACCENT)

    • 7 more.

Use -F or --full-lists to disable shortening of long lists. [code: missing-codepoints]

🔥 FAIL: Check variable font instances (com.google.fonts/check/fvar_instances)
  • 🔥 FAIL fvar instances are incorrect:
  • Add missing instances
Name current expected
ExtraLight Italic N/A wght=200.0, slnt=-11.0
ExtraLight wght=200.0, slnt=0.0 wght=200.0, slnt=0.0
Light Italic N/A wght=300.0, slnt=-11.0
Light wght=300.0, slnt=0.0 wght=300.0, slnt=0.0
Italic N/A wght=400.0, slnt=-11.0
Regular wght=400.0, slnt=0.0 wght=400.0, slnt=0.0
Medium Italic N/A wght=500.0, slnt=-11.0
Medium wght=500.0, slnt=0.0 wght=500.0, slnt=0.0
SemiBold Italic N/A wght=600.0, slnt=-11.0
SemiBold wght=600.0, slnt=0.0 wght=600.0, slnt=0.0
Bold Italic N/A wght=700.0, slnt=-11.0
Bold wght=700.0, slnt=0.0 wght=700.0, slnt=0.0
ExtraBold Italic N/A wght=800.0, slnt=-11.0
ExtraBold wght=800.0, slnt=0.0 wght=800.0, slnt=0.0
Black Italic N/A wght=900.0, slnt=-11.0
Black wght=900.0, slnt=0.0 wght=900.0, slnt=0.0
WARN: Check copyright namerecords match license file. (com.google.fonts/check/name/license)
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN For now we're still accepting http URLs, but you should consider using https instead.
    [code: http]
WARN: License URL matches License text on name table? (com.google.fonts/check/name/license_url)
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=14] [code: http-in-license-info]
  • WARN For now we're still accepting http URLs, but you should consider using https instead.
    [code: http]
WARN: A static fonts directory with at least two fonts must accompany variable fonts (com.google.fonts/check/repo/vf_has_static_fonts)
  • WARN Please consider adding a subdirectory called "static/" and including in it static font files. [code: missing]
WARN: METADATA.pb: Designers are listed correctly on the Google Fonts catalog? (com.google.fonts/check/metadata/designer_profiles)
  • WARN It seems that Accademia di Belle Arti di Urbino is still not listed on the designers catalog. Please submit a photo and a link to a webpage where people can learn more about the work of this designer/typefoundry. [code: profile-not-found]
WARN: Ensure fonts have ScriptLangTags declared on the 'meta' table. (com.google.fonts/check/meta/script_lang_tags)
  • WARN This font file does not have a 'meta' table. [code: lacks-meta-table]
WARN: Glyph names are all valid? (com.google.fonts/check/valid_glyphnames)
  • WARN The following glyph names may be too long for some legacy systems which may expect a maximum 31-char length limit:
    alefMaksura_alefMaksuraar.fina.alt, yehHamzaabove_yehHamzaabovear.fina, alefMaksura_noonar.fina.alt.color0, alefMaksura_noonar.fina.alt.color1, noon_yehHamzaabovear.fina.color0, noon_yehHamzaabovear.fina.color1, noon_yehHamzaabovear.fina.color2, seen_yehHamzaabovear.fina.color0, seen_yehHamzaabovear.fina.color1, sheen_yehHamzaabovear.fina.color0 and 13 more.

Use -F or --full-lists to disable shortening of long lists. [code: legacy-long-names]

WARN: Check font contains no unreachable glyphs (com.google.fonts/check/unreachable_glyphs)
  • WARN The following glyphs could not be reached by codepoint or substitution rules:

    • .notdef.color0

    • AEacute.color0

    • AEacute.color1

    • Aacute.color0

    • Aacute.color1

    • Abreve.color0

    • Abreve.color1

    • Acircumflex.color0

    • Acircumflex.color1

    • Adieresis.color0

    • 1052 more.

Use -F or --full-lists to disable shortening of long lists.
[code: unreachable-glyphs]

WARN: Does the font contain a soft hyphen? (com.google.fonts/check/soft_hyphen)
  • WARN This font has a 'Soft Hyphen' character. [code: softhyphen]
WARN: Ensure dotted circle glyph is present and can attach marks. (com.google.fonts/check/dotted_circle)
  • WARN No dotted circle glyph present [code: missing-dotted-circle]
WARN: Detect any interpolation issues in the font. (com.google.fonts/check/interpolation_issues)
  • WARN Interpolation issues were found in the font: - Contour 0 start point differs in glyph 'Oslash' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a36b8e0> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a3c2680>

    • Contour 0 start point differs in glyph 'Oslash' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a36b8e0> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a3c3df0>

    • Contour 0 start point differs in glyph 'Oslash' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a36b8e0> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a3c1cc0>

    • Contour 0 start point differs in glyph 'Oslashacute' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a36b8e0> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a3c2680>

    • Contour 2 start point differs in glyph 'Oslashacute' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a36b8e0> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a3c2680>

    • Contour 0 start point differs in glyph 'Oslashacute' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a36b8e0> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a3c3df0>

    • Contour 2 start point differs in glyph 'Oslashacute' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a36b8e0> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a3c3df0>

    • Contour 0 start point differs in glyph 'Oslashacute' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a36b8e0> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a3c1cc0>

    • Contour 2 start point differs in glyph 'Oslashacute' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a36b8e0> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a3c1cc0>

    • Contour 0 start point differs in glyph 'uni0686' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a36b8e0> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f996a36be80>

    • 234 more.

Use -F or --full-lists to disable shortening of long lists. [code: interpolation-issues]

WARN: Check GDEF mark glyph class doesn't have characters that are not marks. (com.google.fonts/check/gdef_non_mark_chars)
  • WARN The following non-mark characters should not be in the GDEF mark glyph class:
    U+FBB2, U+FBB3, U+FBB4, U+FBB5, U+FBB6, U+FBB7, U+FBB8, U+FBB9, U+FBBD and U+FBBE [code: non-mark-chars]
INFO: Does DESCRIPTION file contain a upstream Git repo URL? (com.google.fonts/check/description/git_url)
INFO: Show hinting filesize impact. (com.google.fonts/check/hinting_impact)
  • INFO Hinting filesize impact:
ofl/cairo/Cairo[slnt,wght].ttf
Dehinted Size 585.5kb
Hinted Size 585.5kb
Increase 24 bytes
Change 0.0 %
[code: size-impact]
INFO: Font has old ttfautohint applied? (com.google.fonts/check/old_ttfautohint)
  • INFO Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 3.130;gftools[0.9.24]'] [code: version-not-detected]
INFO: EPAR table present in font? (com.google.fonts/check/epar)
INFO: Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? (com.google.fonts/check/gasp)
  • INFO These are the ppm ranges declared on the gasp table:

PPM <= 65535:
flag = 0x0F
- Use grid-fitting
- Use grayscale rendering
- Use gridfitting with ClearType symmetric smoothing
- Use smoothing along multiple axes with ClearType®
[code: ranges]

INFO: Familyname must be unique according to namecheck.fontdata.com (com.google.fonts/check/fontdata_namecheck)
INFO: Check for font-v versioning. (com.google.fonts/check/fontv)
  • INFO Version string is: "Version 3.130;gftools[0.9.24]"
    The version string must ideally include a git commit hash and either a "dev" or a "release" suffix such as in the example below:
    "Version 1.3; git-0d08353-release" [code: bad-format]
INFO: Font contains all required tables? (com.google.fonts/check/required_tables)
  • INFO This font contains the following optional tables:

    • loca

    • prep

    • GPOS

    • GSUB

    • gasp [code: optional-tables]

INFO: List all superfamily filepaths (com.google.fonts/check/superfamily/list)
  • INFO ofl/cairo [code: family-path]

Summary

💔 ERROR 🔥 FAIL ⚠ WARN 💤 SKIP ℹ INFO 🍞 PASS 🔎 DEBUG
0 3 11 41 10 177 0
0% 1% 5% 17% 4% 73% 0%

Note: The following loglevels were omitted in this report:

  • SKIP
  • PASS
  • DEBUG

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 6, 2023

Fontbakery report

Fontbakery version: 0.8.11

[1] Family checks
INFO: Check axis ordering on the STAT table. (com.google.fonts/check/STAT/axis_order)
  • INFO From a total of 1 font files, 0 of them (0.00%) lack a STAT table.

    And these are the most common STAT axis orderings:
    ('slnt-wght', 1) [code: summary]


[21] CairoPlay[slnt,wght].ttf
🔥 FAIL: Check Google Fonts glyph coverage. (com.google.fonts/check/glyph_coverage)
  • 🔥 FAIL Missing required codepoints:

    • 0x0308 (COMBINING DIAERESIS)

    • 0x0300 (COMBINING GRAVE ACCENT)

    • 0x0301 (COMBINING ACUTE ACCENT)

    • 0x030B (COMBINING DOUBLE ACUTE ACCENT)

    • 0x0304 (COMBINING MACRON)

    • 0x0132 (LATIN CAPITAL LIGATURE IJ)

    • 0x1E9E (LATIN CAPITAL LETTER SHARP S)

    • 0x0133 (LATIN SMALL LIGATURE IJ)

    • 0x0307 (COMBINING DOT ABOVE)

    • 0x0302 (COMBINING CIRCUMFLEX ACCENT)

    • 7 more.

Use -F or --full-lists to disable shortening of long lists. [code: missing-codepoints]

🔥 FAIL: Check variable font instances (com.google.fonts/check/fvar_instances)
  • 🔥 FAIL fvar instances are incorrect:
  • Add missing instances
Name current expected
ExtraLight Italic N/A wght=200.0, slnt=-11.0
ExtraLight wght=200.0, slnt=0.0 wght=200.0, slnt=0.0
Light Italic N/A wght=300.0, slnt=-11.0
Light wght=300.0, slnt=0.0 wght=300.0, slnt=0.0
Italic N/A wght=400.0, slnt=-11.0
Regular wght=400.0, slnt=0.0 wght=400.0, slnt=0.0
Medium Italic N/A wght=500.0, slnt=-11.0
Medium wght=500.0, slnt=0.0 wght=500.0, slnt=0.0
SemiBold Italic N/A wght=600.0, slnt=-11.0
SemiBold wght=600.0, slnt=0.0 wght=600.0, slnt=0.0
Bold Italic N/A wght=700.0, slnt=-11.0
Bold wght=700.0, slnt=0.0 wght=700.0, slnt=0.0
ExtraBold Italic N/A wght=800.0, slnt=-11.0
ExtraBold wght=800.0, slnt=0.0 wght=800.0, slnt=0.0
Black Italic N/A wght=900.0, slnt=-11.0
Black wght=900.0, slnt=0.0 wght=900.0, slnt=0.0
WARN: Check copyright namerecords match license file. (com.google.fonts/check/name/license)
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN For now we're still accepting http URLs, but you should consider using https instead.
    [code: http]
WARN: License URL matches License text on name table? (com.google.fonts/check/name/license_url)
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=13] [code: http-in-description]
  • WARN Please consider using HTTPS URLs at name table entry [plat=3, enc=1, name=14] [code: http-in-license-info]
  • WARN For now we're still accepting http URLs, but you should consider using https instead.
    [code: http]
WARN: A static fonts directory with at least two fonts must accompany variable fonts (com.google.fonts/check/repo/vf_has_static_fonts)
  • WARN Please consider adding a subdirectory called "static/" and including in it static font files. [code: missing]
WARN: METADATA.pb: Designers are listed correctly on the Google Fonts catalog? (com.google.fonts/check/metadata/designer_profiles)
  • WARN It seems that Accademia di Belle Arti di Urbino is still not listed on the designers catalog. Please submit a photo and a link to a webpage where people can learn more about the work of this designer/typefoundry. [code: profile-not-found]
WARN: Ensure fonts have ScriptLangTags declared on the 'meta' table. (com.google.fonts/check/meta/script_lang_tags)
  • WARN This font file does not have a 'meta' table. [code: lacks-meta-table]
WARN: Glyph names are all valid? (com.google.fonts/check/valid_glyphnames)
  • WARN The following glyph names may be too long for some legacy systems which may expect a maximum 31-char length limit:
    alefMaksura_alefMaksuraar.fina.alt, yehHamzaabove_yehHamzaabovear.fina, alefMaksura_noonar.fina.alt.color0, alefMaksura_noonar.fina.alt.color1, noon_yehHamzaabovear.fina.color0, noon_yehHamzaabovear.fina.color1, noon_yehHamzaabovear.fina.color2, seen_yehHamzaabovear.fina.color0, seen_yehHamzaabovear.fina.color1, sheen_yehHamzaabovear.fina.color0 and 13 more.

Use -F or --full-lists to disable shortening of long lists. [code: legacy-long-names]

WARN: Check font contains no unreachable glyphs (com.google.fonts/check/unreachable_glyphs)
  • WARN The following glyphs could not be reached by codepoint or substitution rules:

    • at.cap

    • braceleft.cap

    • braceright.cap

    • bracketleft.cap

    • bracketright.cap

    • connection_r

    • dotcenterar

    • eight.pnum

    • emdash.cap

    • endash.cap

    • 20 more.

Use -F or --full-lists to disable shortening of long lists.
[code: unreachable-glyphs]

WARN: Does the font contain a soft hyphen? (com.google.fonts/check/soft_hyphen)
  • WARN This font has a 'Soft Hyphen' character. [code: softhyphen]
WARN: Ensure dotted circle glyph is present and can attach marks. (com.google.fonts/check/dotted_circle)
  • WARN No dotted circle glyph present [code: missing-dotted-circle]
WARN: Detect any interpolation issues in the font. (com.google.fonts/check/interpolation_issues)
  • WARN Interpolation issues were found in the font: - Contour 0 start point differs in glyph 'Oslash' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75c5030> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75a9540>

    • Contour 0 start point differs in glyph 'Oslash' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75c5030> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75aada0>

    • Contour 0 start point differs in glyph 'Oslash' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75c5030> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75a8220>

    • Contour 0 start point differs in glyph 'Oslashacute' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75c5030> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75a9540>

    • Contour 2 start point differs in glyph 'Oslashacute' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75c5030> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75a9540>

    • Contour 0 start point differs in glyph 'Oslashacute' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75c5030> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75aada0>

    • Contour 2 start point differs in glyph 'Oslashacute' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75c5030> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75aada0>

    • Contour 0 start point differs in glyph 'Oslashacute' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75c5030> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75a8220>

    • Contour 2 start point differs in glyph 'Oslashacute' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75c5030> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75a8220>

    • Contour 0 start point differs in glyph 'uni0686' between location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75c5030> and location <fontTools.ttLib.ttGlyphSet._TTGlyphSetGlyf object at 0x7f3ee75c5060>

    • 234 more.

Use -F or --full-lists to disable shortening of long lists. [code: interpolation-issues]

WARN: Check GDEF mark glyph class doesn't have characters that are not marks. (com.google.fonts/check/gdef_non_mark_chars)
  • WARN The following non-mark characters should not be in the GDEF mark glyph class:
    U+FBB2, U+FBB3, U+FBB4, U+FBB5, U+FBB6, U+FBB7, U+FBB8, U+FBB9, U+FBBD and U+FBBE [code: non-mark-chars]
INFO: Does DESCRIPTION file contain a upstream Git repo URL? (com.google.fonts/check/description/git_url)
INFO: Show hinting filesize impact. (com.google.fonts/check/hinting_impact)
  • INFO Hinting filesize impact:
ofl/cairoplay/CairoPlay[slnt,wght].ttf
Dehinted Size 593.9kb
Hinted Size 593.9kb
Increase 24 bytes
Change 0.0 %
[code: size-impact]
INFO: Font has old ttfautohint applied? (com.google.fonts/check/old_ttfautohint)
  • INFO Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 3.130;gftools[0.9.24]'] [code: version-not-detected]
INFO: EPAR table present in font? (com.google.fonts/check/epar)
INFO: Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? (com.google.fonts/check/gasp)
  • INFO These are the ppm ranges declared on the gasp table:

PPM <= 65535:
flag = 0x0F
- Use grid-fitting
- Use grayscale rendering
- Use gridfitting with ClearType symmetric smoothing
- Use smoothing along multiple axes with ClearType®
[code: ranges]

INFO: Check for font-v versioning. (com.google.fonts/check/fontv)
  • INFO Version string is: "Version 3.130;gftools[0.9.24]"
    The version string must ideally include a git commit hash and either a "dev" or a "release" suffix such as in the example below:
    "Version 1.3; git-0d08353-release" [code: bad-format]
INFO: Font contains all required tables? (com.google.fonts/check/required_tables)
  • INFO This font contains the following optional tables:

    • loca

    • prep

    • GPOS

    • GSUB

    • gasp [code: optional-tables]

INFO: List all superfamily filepaths (com.google.fonts/check/superfamily/list)
  • INFO ofl/cairoplay [code: family-path]

Summary

💔 ERROR 🔥 FAIL ⚠ WARN 💤 SKIP ℹ INFO 🍞 PASS 🔎 DEBUG
0 2 11 41 9 179 0
0% 1% 5% 17% 4% 74% 0%

Note: The following loglevels were omitted in this report:

  • SKIP
  • PASS
  • DEBUG

Comment thread ofl/cairo/METADATA.pb
license: "OFL"
category: "DISPLAY"
category: "SANS_SERIF"
category: "DISPLAY"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Display category should remain first in the list (secondary category)

@yanone yanone closed this Mar 6, 2023
@yanone yanone deleted the gftools_packager_ofl_cairo branch March 6, 2023 16:49
@yanone yanone removed a link to an issue Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

I Font Bug III Improve rendering / layout by addition of manual hinting, kerning or else

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants