-
Notifications
You must be signed in to change notification settings - Fork 13
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
"Oops, this parser is just broken" #801
Comments
thanks. Could you try with |
actually RUST_BACKTRACE=1 would not work in this case as this is not a panic. RUST_LOG=debug maybe |
I put a few println! before the SomethingWentWrong error which gives the Ooops, and I think it's the "disabled = 1" key in some of the custom parameters that is making it fail. So the glyphs-reader parser currently doesn't support disabled custom parameters. Looks like it's a known issue #312 If I remove those disabled custom params, I get another error but probably better to file a separate issue for that. |
ideally we should support parsing disabled custom params, but at least this generic error message should be made clearer, more specific |
Thank you for testing! |
Thank you for the report |
this is fixed in #880 |
I tried to compile my Sudo font with
fontc
, but it fails without any specifics:I converted the original
.glyphspackage
file to.glyphs
, attached.Sudo.glyphs.zip
The text was updated successfully, but these errors were encountered: