search

silnrsi / smith

font development, testing and release
Other
14 stars 5 forks source link

integrate ttffautohint (including parameters) #44

Open n7s opened 6 years ago

n7s commented 6 years ago

please integrate ttfautohint into smith, with the ability to indicate the various parameters.

How about a font parameter of: ttfautohint_parms="-n -W" for example?

nrsiward commented 6 years ago

Example of how to integrate into wscript:

font(target = process(outfile, ...
            cmd("${TTFAUTOHINT} -n -W ${DEP} ${TGT}")),
    …
     )
def configure(ctx):
        ctx.find_program('ttfautohint')

We need to add a parameter to the font class so the above is not needed.

mhosken commented 6 years ago

Do we want ttfautohint to be run even if there is no ttfautohint_parms? If not, then perhaps we should just call the attribute ttfautohint? Do we make smith dependent on ttfautohint or do we only run it if configure manages to find it?

nrsiward commented 6 years ago

There was no discussion at TM2018 about what to call the parameter to smith. I think ttfautohint sounds good and smith should only run ttfautohint if that parameter is present.

I don't want to make smith dependent on ttfautohint since it's unclear how many projects will use it. If configure cannot find it and the parameter is present, will it issue a clear warning? If so, then that's probably all that is needed.