perrette / papers

Command-line tool to manage bibliography (pdfs + bibtex)
MIT License
142 stars 22 forks source link

fix: Does not install dependencies #25

Closed aiotter closed 2 years ago

aiotter commented 2 years ago

There is no such argument as requires in distutils.core.setup() (see docs). That is why executing pip install papers-cli does not install dependencies such as six, bibtexparser, etc. This PR fixes this.

As you can see here, disutils package is deprecated and should not be used. That's why I suggest to switch from disutils to setuptools. Furthermore, there is no dependency resolving feature in disutils afaik.

perrette commented 2 years ago

Thanks much, agreed. Let's switch then.