-
Notifications
You must be signed in to change notification settings - Fork 442
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
Python3 port #267
Python3 port #267
Conversation
Can confirm this builds NotoColorEmoji.ttf fine! As mentioned, currently this depends on this specific nototools branch. Lets push for that to be merged before merging this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this! I left some comments below. Let me know if you have time to address these or I can take this up.
@mike-fabian hey! thanks again for patch. I'll merge and apply the fix-ups myself |
78c5873
to
60161a3
Compare
Cosimo Lupo <notifications@github.com> さんは書きました:
@mike-fabian hey! thanks again for patch. I'll merge and apply the fix-ups myself
Thank you!
…--
📧 Mike FABIAN <mike.fabian@gmx.de>
睡眠不足はいい仕事の敵だ。
|
Code looks good, font is being built without a hitch. 👍 |
Hi! It would help Debian switch their nototools from python2 to python3 if someone would also port https://github.com/googlefonts/robotoslab |
@jbicha I wasn't aware that project relies on python2-only code. Can you please open an issue over there pointing to the files that need update? Thanks |
@anthrotype Never mind. You're right. It was just a copy-and-paste mistake in the Debian packaging for robotoslab. |
Together with the python3 port of nototools
notofonts/nototools#488
This pull request makes it possible to build the font using Python3.
Using Python2 still works as well, the code works with both.