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 autohintexe #606

Merged
merged 1 commit into from
Sep 11, 2018
Merged

Remove autohintexe #606

merged 1 commit into from
Sep 11, 2018

Conversation

miguelsousa
Copy link
Member

We currently get this tool via psautohint v1.8.1

We currently get this tool via psautohint v1.8.1
@miguelsousa miguelsousa merged commit c405ac3 into develop Sep 11, 2018
@miguelsousa miguelsousa deleted the remove-autohintexe branch September 11, 2018 21:23
@Arnie97
Copy link
Contributor

Arnie97 commented Oct 18, 2018

Hi, I got an error message after the 2.8.2 release:

Please re-install the FDK. The path to the program 'autohintexe' is not in the environment variable PATH.

This seems to be a Windows-specific issue -- the same release works well on my Linux box. Neither platform has autohintexe in its $PATH.

@miguelsousa
Copy link
Member Author

That message seems to be coming from this line, https://github.com/adobe-type-tools/afdko/blob/develop/python/afdko/autohint.py#L556

The Windows CI is not failing, so I'm not sure how to reproduce your problem.
Did you have a previous AFDKO installed?

@Arnie97
Copy link
Contributor

Arnie97 commented Oct 18, 2018

Well, this was caused by spaces in the psautohint.AUTOHINTEXE path...

@miguelsousa
Copy link
Member Author

@Arnie97 that shouldn't happen. I'd like to fix this. Where exactly are the spaces causing the problem?

miguelsousa added a commit that referenced this pull request Mar 6, 2019
The autohint source files were removed in #606
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants