enthought / enable

Enable: low-level drawing and interaction
Other
91 stars 45 forks source link

Savage depends on `pyparsing` but not directly in dependencies #1040

Closed corranwebster closed 1 year ago

corranwebster commented 1 year ago

It likely gets pulled in by other dependencies; we should make it explicit, however.

mdickinson commented 1 year ago

See also #543, and enthought/chaco#858

corranwebster commented 1 year ago

There is now an optional "svg" dependency which requires pyparsing, and this is now required by Chaco, so I think that this is resolved.

mdickinson commented 1 year ago

@corranwebster Should #543 also be closed?

corranwebster commented 1 year ago

Not sure - "test" doesn't include pyparsing - let me check

corranwebster commented 1 year ago

Nope - still happens, and there's now also an error if celiagg isn't installed.