-
-
Notifications
You must be signed in to change notification settings - Fork 29
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
Rebuild with system harfbuzz #102
Rebuild with system harfbuzz #102
Conversation
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
@conda-forge-admin please rerender |
Looking at the build logs from #84 (a faster build) it seems that system harfbuzz is being found and used. Was there any particular reason to not enable harfbuzz on all platforms for Qt? |
a7c2a90
to
eaed471
Compare
…nda-forge-pinning 2023.01.16.10.03.29
eaed471
to
a35aed3
Compare
I don't know if it's really necessary on Mac because it probably has its own font shaping engine. In this regard I'd follow what the Homebrew Qt recipe does. |
Ok i think i understand (sorry this is about the 5th time that you explain to me that OSX does not need harfbuzz) |
Maybe the correct solution is to use system harfbuzz only on linux. |
@conda-forge-admin please rerender |
…nda-forge-pinning 2023.01.18.19.16.13
(so strange that I can't rerender locally) |
Any final thoughts? |
If aarch and pcc64le builds are important, please open an issue and build them in accordance to CFEP03 and I can upload. |
Looking at the build logs from #84 (a faster build) it seems that system harfbuzz is being found and used.
Was there any particular reason to not enable harfbuzz on all platforms for Qt?
Closes #101.
Here's a checklist to do before merging.