Runtime dependency on setuptools
Bug #303938 reported by
Philippe Normand
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Moovida |
Fix Released
|
Medium
|
Philippe Normand |
Bug Description
This is a code/packaging issue. Not trivial to reproduce and relevant most likely only for packagers.
Elisa doesn't need to import setuptools at runtime, it's useful only during the build process (when installing elisa and build tarballs).
The plugin_registry imports elisa.core.dist (which imports setuptools). After quick investigation, only 2 functions of dist.py are used by the plugin_registry and they don't rely on setuptools package, which is good. So what we could do is:
- either move those 2 functions (only used by the plugin_registry) to the plugin_registry.py file
- or move them to another (new?) module
Changed in elisa: | |
importance: | Undecided → Medium |
milestone: | none → 0.5.x |
status: | New → Confirmed |
Changed in elisa: | |
assignee: | nobody → philn |
milestone: | 0.5.x → 0.5.21 |
status: | Confirmed → Fix Committed |
status: | Fix Committed → In Progress |
Changed in elisa: | |
status: | In Progress → Fix Committed |
Changed in elisa: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
The Elisa Pigment plugin also imports elisa.core.dist (in pigment_ frontend. py) to parse the "mappings" files