Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jost: v3.51 added #2627

Merged
merged 5 commits into from
Sep 1, 2020
Merged

jost: v3.51 added #2627

merged 5 commits into from
Sep 1, 2020

Conversation

EwonRael
Copy link
Contributor

@m4rc1e
Copy link
Collaborator

m4rc1e commented Aug 26, 2020

Thanks for PR. Can you please do the following:

  • Run the VF through gftools fix-vf-meta
  • Bump the version number (gf has 3.6, yet this font has 3.5). I expect > 3.6.

Any reason why the italic angle has changed from -12 to -10?

@m4rc1e
Copy link
Collaborator

m4rc1e commented Aug 26, 2020

Fontbakery report

Fontbakery version: 0.7.29

[1] Family checks
WARN: Is the command `ftxvalidator` (Apple Font Tool Suite) available?
--- Rationale ---

There's no reasonable (and legal) way to run the command `ftxvalidator` of the
Apple Font Tool Suite on a non-macOS machine. I.e. on GNU+Linux or Windows etc.

If Font Bakery is not running on an OSX machine, the machine running Font
Bakery could access `ftxvalidator` on OSX, e.g. via ssh or a remote procedure
call (rpc).

There's an ssh example implementation at:
https://github.com/googlefonts/fontbakery/blob/master/prebuilt/workarounds
/ftxvalidator/ssh-implementation/ftxvalidator


  • WARN Could not find ftxvalidator.

[3] Jost-Italic[wght].ttf
WARN: Check copyright namerecords match license file.
--- Rationale ---

A known licensing description must be provided in the NameID 14 (LICENSE
DESCRIPTION) entries of the name table.

The source of truth for this check (to determine which license is in use) is a
file placed side-by-side to your font project including the licensing terms.

Depending on the chosen license, one of the following string snippets is
expected to be found on the NameID 13 (LICENSE DESCRIPTION) entries of the name
table:
- "This Font Software is licensed under the SIL Open Font License, Version 1.1.
This license is available with a FAQ at: https://scripts.sil.org/OFL"
- "Licensed under the Apache License, Version 2.0"
- "Licensed under the Ubuntu Font Licence 1.0."


Currently accepted licenses are Apache or Open Font License.
For a small set of legacy families the Ubuntu Font License may be acceptable as
well.

When in doubt, please choose OFL for new font projects.


  • 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?
--- Rationale ---

A known license URL must be provided in the NameID 14 (LICENSE INFO URL) entry
of the name table.

The source of truth for this check is the licensing text found on the NameID 13
entry (LICENSE DESCRIPTION).

The string snippets used for detecting licensing terms are:
- "This Font Software is licensed under the SIL Open Font License, Version 1.1.
This license is available with a FAQ at: https://scripts.sil.org/OFL"
- "Licensed under the Apache License, Version 2.0"
- "Licensed under the Ubuntu Font Licence 1.0."


Currently accepted licenses are Apache or Open Font License.
For a small set of legacy families the Ubuntu Font License may be acceptable as
well.

When in doubt, please choose OFL for new font projects.


  • 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: Check mark characters are in GDEF mark glyph class
--- Rationale ---

Mark characters should be in the GDEF mark glyph class.


  • WARN The following mark characters could be in the GDEF mark glyph class:
    U+0300, U+0301, U+0302, U+0303, U+0304, U+0306, U+0307, U+0308, U+030A, U+030B, U+030C, U+0312, U+0326, U+0327 and U+0328 [code: mark-chars]

[3] Jost[wght].ttf
WARN: Check copyright namerecords match license file.
--- Rationale ---

A known licensing description must be provided in the NameID 14 (LICENSE
DESCRIPTION) entries of the name table.

The source of truth for this check (to determine which license is in use) is a
file placed side-by-side to your font project including the licensing terms.

Depending on the chosen license, one of the following string snippets is
expected to be found on the NameID 13 (LICENSE DESCRIPTION) entries of the name
table:
- "This Font Software is licensed under the SIL Open Font License, Version 1.1.
This license is available with a FAQ at: https://scripts.sil.org/OFL"
- "Licensed under the Apache License, Version 2.0"
- "Licensed under the Ubuntu Font Licence 1.0."


Currently accepted licenses are Apache or Open Font License.
For a small set of legacy families the Ubuntu Font License may be acceptable as
well.

When in doubt, please choose OFL for new font projects.


  • 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?
--- Rationale ---

A known license URL must be provided in the NameID 14 (LICENSE INFO URL) entry
of the name table.

The source of truth for this check is the licensing text found on the NameID 13
entry (LICENSE DESCRIPTION).

The string snippets used for detecting licensing terms are:
- "This Font Software is licensed under the SIL Open Font License, Version 1.1.
This license is available with a FAQ at: https://scripts.sil.org/OFL"
- "Licensed under the Apache License, Version 2.0"
- "Licensed under the Ubuntu Font Licence 1.0."


Currently accepted licenses are Apache or Open Font License.
For a small set of legacy families the Ubuntu Font License may be acceptable as
well.

When in doubt, please choose OFL for new font projects.


  • 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: Check mark characters are in GDEF mark glyph class
--- Rationale ---

Mark characters should be in the GDEF mark glyph class.


  • WARN The following mark characters could be in the GDEF mark glyph class:
    U+0300, U+0301, U+0302, U+0303, U+0304, U+0306, U+0307, U+0308, U+030A, U+030B, U+030C, U+0312, U+0326, U+0327 and U+0328 [code: mark-chars]

Summary

💔 ERROR 🔥 FAIL ⚠ WARN 💤 SKIP ℹ INFO 🍞 PASS 🔎 DEBUG
0 0 7 74 17 242 0
0% 0% 2% 22% 5% 71% 0%

Note: The following loglevels were omitted in this report:

  • SKIP
  • INFO
  • PASS
  • DEBUG

Diff images: jost_qa.zip

@EwonRael
Copy link
Contributor Author

EwonRael commented Aug 26, 2020

I have made the necessary changes

italic angle of -10 is more reflective of the actual angle in the font.

@m4rc1e
Copy link
Collaborator

m4rc1e commented Aug 26, 2020

@EwonRael can you add these changes to your upstream build script please? I see there hasn't been an update in six days.

@EwonRael
Copy link
Contributor Author

are you looking at google fonts branch?

https://github.com/indestructible-type/Jost/tree/google

I will update master in a min but google branch is better place to pull from

@EwonRael
Copy link
Contributor Author

sorry.. should I have put https://github.com/indestructible-type/Jost/tree/google as upstream repo? I'm new at all this

@thlinard
Copy link
Contributor

fsSelection bit 7 should be enabled (fonttools/fontbakery#3021).

@EwonRael
Copy link
Contributor Author

does it look good now? I have made the updates to the fsSelection bit

@m4rc1e
Copy link
Collaborator

m4rc1e commented Sep 1, 2020

@EwonRael Thanks! LGTM

@m4rc1e m4rc1e merged commit 9592afd into google:master Sep 1, 2020
@davelab6
Copy link
Member

davelab6 commented Sep 3, 2020

@EwonRael will you merge the google branch to main?

@EwonRael
Copy link
Contributor Author

EwonRael commented Sep 3, 2020

both google and main should be up to date on repo?

davelab6 added a commit that referenced this pull request Sep 9, 2020
Jost #2627, Rubik #2664, Sansita Swashed #2658, Piazzolla #2653, Baskervville Epilogue Grandstander #2648, Merriweather #2632, Syne #2644
@RosaWagner RosaWagner added to_production --- Live Font is visible on API and removed in_sandbox labels Sep 23, 2020
davelab6 pushed a commit that referenced this pull request Dec 10, 2020
* jost: v3.51 added

* Jost 3.7

* fsSelection bit 7
davelab6 added a commit that referenced this pull request Dec 10, 2020
Jost #2627, Rubik #2664, Sansita Swashed #2658, Piazzolla #2653, Baskervville Epilogue Grandstander #2648, Merriweather #2632, Syne #2644
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
--- Live Font is visible on API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants