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

Remove autohint tool #826

Closed
8 tasks done
miguelsousa opened this issue Jun 27, 2019 · 0 comments · Fixed by #1175
Closed
8 tasks done

Remove autohint tool #826

miguelsousa opened this issue Jun 27, 2019 · 0 comments · Fixed by #1175
Assignees
Labels
high priority important but not urgent to be deprecated

Comments

@miguelsousa
Copy link
Member

miguelsousa commented Jun 27, 2019

Remove autohint tool in favor of psautohint.

  • add tests to validate that psautohint produces the same output as autohint. This will require some adjustment to autohint and test code:
    • update autohint to output PFB when PFB is supplied as input (currently outputs PFA). See f25415a
    • add method in test_utils.py to use fontTools.cffLib.specializer, to enable comparison of equivalent Type2 Charstrings in CFF-based output (currently, output from autohint is optimized, while output from psautohint is not).
    • skip comparisons where non-default-layer UFO2 is used, since psautohint can only write to the default layer for UFO2.
    • fix bug in autohint where it does not delete com.adobe.processedgyphs layer if present when run with the -wd option. Fix should probably go here: https://github.com/adobe-type-tools/afdko/blob/develop/python/afdko/autohint.py#L1036
  • replace content in autohint.py that simply calls psautohint
  • add FutureWarnings in above
  • Remove autohint.py and related test code from AFDKO [early 2020]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high priority important but not urgent to be deprecated
Projects
None yet
3 participants