Include a python3 version of the font-patcher script

[SageEx]

Include a python3 version of the font-patcher script

[ryanoasis]

@SageEx Thanks for the info and I am glad this project is useful to you 😄

python 3

I will have to look into that, I see the changes you had to make. There has to be a way to make it compatible for py3 and py2 without having 2 separate scripts to maintain?

how about including Fantasque Sans Mono in the default patched fonts set ?

Looks like the license will allow this (OFL (SIL Open Font License)), the only I was checking for is if there is a Reserved Font Name specified in the license but I don't see one. If the name is reserved we would just have to rename the patched version (e.g. Hack becomes Knack).

[SageEx]

Most of the changes are due to the convention changes from python2 to python3 - like

• brackets in print statements
• dic.iteritems() to dic.items
  etc.

I doubt that a single python script would suffice.

Rather, as a solution, you can maintain the python2 script only and use 2to3 to automatically generate the python3 script. The only manual change you will have to make in the generated script is to revert the change at line 306. By default 2to3 will change from Unicode to str - but str is not an attribute - hence the error. Any way to use 2to3 which does not change this will be the answer to this problem.

I really hope to see the Fantasque Sans Mono font in the repo.

Thanks again for this repo - along with vim-webdevicons.

[ryanoasis]

Thanks btw. I will probably get to this but not anytime soon.

Fantasque Sans Mono

Probably not too difficult to add just a handful of bugs that I've been focusing on.

Cheers!

[ryanoasis]

I don't have fontforge setup with py3 bindings nor have it compiled with py3 so I cannot test however this script: font-patcher-py3 should work for python3, if someone could test it out? That'd be great!

fontforge -script font-patcher-py3 unpatched-sample-fonts/Monoid/Monoid-Italic.ttf --powerline --powerlineextra
or
python3 font-patcher-py3 unpatched-sample-fonts/Monoid/Monoid-Italic.ttf --powerline --powerlineextra

This is only on the 0.7.0 branch for now.

I will test it when I get a chance.

[github-actions]

This issue has been automatically locked since there has not been any recent activity (i.e. last half year) after it was closed. It helps our maintainers focus on the active issues. If you have found a problem that seems similar, please open a new issue, complete the issue template with all the details necessary to reproduce, and mention this issue as reference.