https://codereview.appspot.com/553580043/diff/571780045/mf/GNUmakefile File mf/GNUmakefile (right):
https://codereview.appspot.com/553580043/diff/571780045/mf/GNUmakefile#newcode130 mf/GNUmakefile:130: $(outdir)/emmentaler-brace.otf: emmentaler-brace.fontforge.py $(outdir)/emmentaler-brace.otf-table \ On 2020/02/29 21:33:10, lemzwerg wrote: > maybe break line to make it shorter... Acknowledged. https://codereview.appspot.com/553580043/diff/571780045/mf/emmentaler-brace.fontforge.py File mf/emmentaler-brace.fontforge.py (right): https://codereview.appspot.com/553580043/diff/571780045/mf/emmentaler-brace.fontforge.py#newcode1 mf/emmentaler-brace.fontforge.py:1: #!@FONTFORGE@ On 2020/02/29 21:33:11, lemzwerg wrote: > Can we assume that FontForge's python support and is always enabled? Shall we > check this? the FF page doesn't say that python is optional. https://codereview.appspot.com/553580043/diff/571780045/mf/emmentaler-brace.fontforge.py#newcode6 mf/emmentaler-brace.fontforge.py:6: import fontforge On 2020/02/29 21:33:11, lemzwerg wrote: > importing 'fontforge' twice is correct? If yes, please add a comment. Done. https://codereview.appspot.com/553580043/diff/571780045/mf/gen-emmentaler.fontforge.py File mf/gen-emmentaler.fontforge.py (right): https://codereview.appspot.com/553580043/diff/571780045/mf/gen-emmentaler.fontforge.py#newcode63 mf/gen-emmentaler.fontforge.py:63: alphabet ="feta-alphabet%(design_size)d" % vars() On 2020/02/29 21:33:11, lemzwerg wrote: > whitespace Done. https://codereview.appspot.com/553580043/
