Dependencies missing from setup.py

ckan / ckanext-harvest

Remote harvesting extension for CKAN

130 stars 203 forks source link

Dependencies missing from setup.py #465

Open frafra opened 3 years ago

frafra commented 3 years ago

setup.py is broken at the moment.

https://github.com/ckan/ckanext-harvest/blob/e2e8acbb6d87d44663e92c5ff5cbd67401588c2d/setup.py#L21-L24

Couldn't we have the dependencies listed in setup.py and use . in requirements.txt or just read requirements lines in setup.py to avoid duplication and having something that can be installed using pip?

frafra commented 3 years ago

ckanext-scheming does not suffer of the same issue: https://github.com/ckan/ckanext-scheming/blob/a5b5522302a3cabb94f5bdb1a30a3bb7a5133403/setup.py#L27-L33