-
Notifications
You must be signed in to change notification settings - Fork 70
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
pip3 install gftools doesn't work #393
Comments
The package for the ninja build system failed to compile. Seems to be the "Apple M1" ghost haunting us again. But now on such a central package... a build system package! I bet it will be sorted out quickly. Seems similar to this: fonttools/ttfautohint-py#7 |
Is this on Apple Silicon, @davelab6? I don’t feel like we should be making people build meson and ninja anyway. There should be binary wheels for ots. |
Ah, OK, so there are, but not for M1. There's already an open discussion about whether to ditch ots-python and move to pyots, but the same issue applies. (Although admittedly pyots is further down the road of GitHub actions.) I think there are some good reasons why it would be good to move to pyots and send them a PR to get M1 working. |
according to https://github.com/scikit-build/ninja-python-distributions/releases/tag/1.10.2, there are now universal2 wheels for ninja on PyPI. Can someone try again and confirm that it works now from an M1 mac? |
@simoncozens @eliheuer I updated the ots-python CI to build You |
you should be able to pip install now => https://pypi.org/project/opentype-sanitizer/8.1.4.post3/#files Do let me know if there are other dependencies of gftools that need to be rebuilt for Apple Silicon |
Thank you @anthrotype! I did some testing on my M1 mac (Big Sur 11.2, Python 3.9.6) today.
Thanks again! A Mac Mini M1 is my main computer for working with Glyphs and I have been switching to an old Mac laptop every time I need to run Google Fonts CLI tools, that was a huge pain. |
@eliheuer thanks for checking! |
Thanks everyone! |
thanks! |
The text was updated successfully, but these errors were encountered: