Cannot specify font properties in template but font from CLI options

[sile-typesetter]

Cannot specify font properties in template but font from CLI options

[alerque]

This is about to get marked as closed by PR #40 which I think covers most of this by fixing the bug you hit and adding a feature besides. Uncovered are...

• Whatever your deal was with "font manager". That sounds like SILE itself isn't installed/configured correctly on the system. I'm happy to look into that too but I'll need more information about the installation. Also lets check that sile itself works for normal SILE-ish things and report that on the SILE issue tracker if there is a problem. If sile works by itself and fontproof invocations are fouling it up do open a new issue here with more info. The font manager is something that should be discovered by ./configure time by SILE when it is built or packaged.

• Your question about font family: there is an -F/--family flag for that if you need it, but you only want to specify one of either filename or family, not both. The crash when it was empty was a bug and just about outputting debug information.

• Papersize settings should be inherited from the regular class. I don't see anything at all in FontProof so I can't track down that error. Perhaps the specific project you were testing there needs an update to SILE v0.14.x usage. Can you point me to the template or other files that threw that error? The Andika-Mithani Latin template you linked works fine for me. Open a new issue if you have details.

[alerque]

As you can probably guess from the title, I recognize what this is about. I believe the font being specified from the CLI and from the class-provided command in the template are clobbering each other because they both try to take total control of the font but neither has all the attributes available to do so.

[n7s]

Thanks for the fixes, it works now (having also adjusted some parameters in our integration code).

[alerque]

@n7s The UFO test repository you linked me too early is full of broken stuff so I'm not 100% what your current integration code looks like, but here is one way to do it in GitLab CI: silnrsi/font-andika-mtihani#2. Of course installing the LuaRock in the Ubuntu container would work too instead of using a separate job, but I find modular jobs that stick to a specific task easier to manage in the long run.

[n7s]

The integration changes are not pushed yet (probably next week) but I'm referring to changes needed in the smith repository. I'm going the luarocks installation route. (We are not using GitHub action or Gitlab pipelines at this stage but an existing CI setup already used for a bunch of other SIL software).