neulab/ExplainaBoard

Use setup.cfg to store every setup config

Closed this issue · 2 comments

Current setup.py follows a legacy pattern: writing settings in the code. Recent versions of setuptools supports to store most configs into setup.cfg, and we don't need writing setup "script" anymore.
This can avoid some complicated issues (typically circular imports) around __version__. setup.cfg can import/evaluate version file without any effects against code base.

The only issue here is the installation of en_web_core_sm spaCy package, which is performed at the last step of the setup.py. As I noted in #173, this is not a reliable way, and manually installing this package by users is much better than doing too much by us.

The underlying task in this issue is follows:

  • Move every setting in setup.py to setup.cfg.
  • Remove ad-hoc installation of en_web_core_sm and add documentation about manual installation of it.

Recent versions of distutil supports to store most configs into setup.cfg, and we don't need writing setup "script" anymore.

@odashi I guess setuptools? It seems like distutils is deprecated.

@tetsuok Yes, it was just a typo. Edited.