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

add: smu and graphite2 #358

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

add: smu and graphite2 #358

wants to merge 3 commits into from

Conversation

henriqueleng
Copy link

CHANGES:

  • add graphite and, enable it + icu on harfbuzz (simple stuff, will not bloat the lib)
  • add smu, with a simple patch to fix destination dirs.

@rofl0r
Copy link
Member

rofl0r commented Oct 7, 2015

what's the advantage of using icu and graphite on harfbuzz ?
note that i deliberately disabled icu because it breaks crosscompilation:
http://pkg.sabotage.tech/process.cgi?action=pkgoverview&pkg=icu

@henriqueleng
Copy link
Author

About icu, I enabled it because I'm writing (already did it, just testing) a texlive recipe, and it needs harfbuzz + icu, so I enabled it to extend compatibility.
I didn't know it break crosscompilation, so no problem in removing it, just wondering about a future approach.

@suiginsoft
Copy link

What are you using ICU for?

Harbuzz has a built in version of libUCDN it uses when ICU is disabled, which provides a minimal set of the unicode lookup and transformation functions it needs. You can access them yourself if you look at the hb-unicode.h header.

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.

3 participants