-
Notifications
You must be signed in to change notification settings - Fork 4
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
Expedient plan for moving to read-fonts #13
Comments
Is there discussion somewhere about why this is being managed as a fork instead of working on the Rustybuzz code and gating the ttf-parser interactions behind a feature flag, then adding alternatives gated behind a read-fonts feature flag and requiring one or the other during transition? If that's been considered already, can I at least ask what the current workflow is for keeping the other parts of the code in sync? |
I think it's because originally rustybuzz wasn't part of the harfbuzz org and RazrFalcon didn't transfer ownership back then, so they created a fork instead. And now both kind of exist here.
Also not sure what the official policy is, but I think it would be best to make all changes to rustybuzz for now, and once harfruzz is being worked on full-time, make one big merge from rustybuzz to harfruzz, and from then on focus on just harfruzz (and deprecate rustybuzz once harfruzz is production-ready). |
One thing also to consider is that ideally we want |
The goal is to replace ttf-parser with small diffs that don't break tests. With that in mind, I'm using the following process:
The rationale for this approach is that it’ll be vastly simpler to do cleanup and architectural changes if we’re not dancing around ttf-parser (or worse, mixed ttf-parser/read-fonts) code.
ref comments in #8
The text was updated successfully, but these errors were encountered: