simoncozens / Pendot

Dotter tool for Glyphs
Apache License 2.0
6 stars 0 forks source link

Setting splitPaths = 1 in config data causes build to hang on exec #3

Closed jvgaultney closed 10 months ago

jvgaultney commented 10 months ago

If I try to run a project that has "splitPaths = 1" in the config data in fontinfo.plist the build process hangs on the first exec step. If I set it to "0" it works fine.

simoncozens commented 10 months ago

I can't reproduce this with font-pendot-latin; it builds fine. Can you isolate it to a font with a single glyph and send me that? (By email is OK if you don't want it public.) Flat out hangs are often caused by glyphsLib's .segments method not behaving; ensure you have glyphsLib >=6.6.0 and fontTools>=4.46.0.

jvgaultney commented 10 months ago

To reproduce pull changes to the font-pendot-latin repo, then replace config.yaml with renamed config.yaml.single, which will build from a separate PendotLSingle-Regular.glyphspackage master that contains only the letter E.

jvgaultney commented 10 months ago

For another example of this see failed build #12 vs successful build #13 of font-pendot-latin: https://github.com/silnrsi/font-pendot-latin/actions

simoncozens commented 10 months ago

Finally able to reproduce this and it is indeed yet another bug in GSPath.segments. I will find a minimal reproducer and raise an issue over at glyphsLib.

simoncozens commented 10 months ago

https://github.com/googlefonts/glyphsLib/pull/967 fixes it. You can add

glyphsLib @ git+https://github.com/googlefonts/glyphsLib@segments-again

to requirements.txt if you're impatient...