googlefonts / ufo2ft

A bridge from UFOs to FontTools objects (and therefore, OTFs and TTFs).
MIT License
151 stars 43 forks source link

Remove defcon requirements #557

Closed behdad closed 2 years ago

behdad commented 2 years ago

Either ufoLib2 or defcon is required but not both.

anthrotype commented 2 years ago

No, that file is only used to create the test environment. We run out test suite against both ufoLib2 and defcon to make sure we support both. The actual runtime requirements are listed in setup.py install _requires, only those are used automatically when you do pip install ufo2ft.

anthrotype commented 2 years ago

Of course you need one or the other, but since we don't actually import any of the two directly within ufo2ft itself, it's the responsibility of our client to decide which kind of Font object to pass down to our compile methods

anthrotype commented 2 years ago

I hope that makes sense. Feel free to reopen