Skip to content
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

Ninja error building fonts. ninja: error: build.ninja:57: multiple rules generate #854

Open
juandelperal opened this issue Mar 13, 2024 · 1 comment

Comments

@juandelperal
Copy link

Hi team!

Trying to build fonts I get this error:

(venv) ➜  Montserrat git:(merge-forked-changes) gftools builder sources/config.yaml
ninja: error: build.ninja:57: multiple rules generate ../fonts/variable/Montserrat[wght].ttf

Removed file: /Users/juan/projects/Montserrat/sources/build.ninja
Removed directory: /Users/juan/projects/Montserrat/sources/master_ufo
Done building fonts!
(venv) ➜  Montserrat git:(merge-forked-changes)

(Note that also the message printed is wrong, as it assumes the fonts have been generated: Done building fonts!)

After trying different versions It worked with

(venv) ➜  Montserrat git:(merge-forked-changes) ✗ pip install gftools==0.9.48
[...]
Installing collected packages: gftools
  Attempting uninstall: gftools
    Found existing installation: gftools 0.9.51
    Uninstalling gftools-0.9.51:
      Successfully uninstalled gftools-0.9.51
Successfully installed gftools-0.9.48

(venv) ➜  Montserrat git:(merge-forked-changes) ✗ gftools builder sources/config.yaml
[1/137] fontmake -o ufo --instance-dir instance_ufo -g Montserrat.glyphs
INFO:fontmake.font_project:Building master UFOs and designspace from Glyphs source
INFO:glyphsLib.parser:Parsing .glyphs file
INFO:fontmake.font_project:Saving master_ufo/Montserrat.designspace
INFO:fontmake.font_project:Saving master_ufo/Montserrat-Thin.ufo
INFO:fontmake.font_project:Saving master_ufo/Montserrat-SemiBold.ufo
[...]

Everything works well :)

In these tests I started from an empty venv, installing only gftools.

The files are here in case you wanted to test: https://github.com/JulietaUla/Montserrat/tree/merge-forked-changes

Thanks

@simoncozens
Copy link
Contributor

We have just merged (but not released) a new version of the builder, and I just tested that your font builds fine with that. So you can either go back to 0.9.48 or wait for 1.0.0...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants