PyPI package should depend on pydns

Bug #1661218 reported by Peter Mattern on 2017-02-02
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
pyLibravatar
Undecided
Unassigned

Bug Description

pyLibravatar currently depends on py3dns and pydns to work under Python 3 and Python 2 respectively but PyPI package pyLibravatar does not depend on these.
It would be good if this dependency was fixed.

François Marier (fmarier) wrote :

The dependency is in there: https://git.launchpad.net/pylibravatar/tree/setup.py#n4

Is that not the correct way to do it for PyPI?

Changed in pylibravatar:
status: New → Incomplete
Peter Mattern (pmattern) wrote :

Unfortunately I cannot really judge this, so all I can do is report my findings.
And these are 'pip[N] install pylibravatar' does not pull py[3]dns running the pip implementations belonging to Python 3 or 2 on either Arch Linux or Debian stretch. And so far I haven't seen any such problem with any other PyPI package on those platform which is suggesting the problem is in pyLibravatar, IMHO.

Launchpad Janitor (janitor) wrote :

[Expired for pyLibravatar because there has been no activity for 60 days.]

Changed in pylibravatar:
status: Incomplete → Expired
Changed in pylibravatar:
status: Expired → Triaged
François Marier (fmarier) wrote :

Another article on the difference between setup.py and requirements.txt: https://caremad.io/posts/2013/07/setup-vs-requirement/

frenzy (frenzy-madness) wrote :

Hello.

I also have some troubles because of this bug.

Actually, it doesn't matter if you define your dependencies in setup.py directly or you will read them from requirements file. The important part is to define all dependencies which should be installed together with pylibravatar in setup.py with install_requires.

As you can see in setuptools docs [0], requires kwarg is only for metadata and it doesn't make setuptools to do anything different. What you need to specify is install_requires kwarg as mentioned in the docs as well [1].

So, to fix this problem, change `requires = requires,` to `install_requires = requires,` in setup.py on line 19.

Could you please try this fix and release a new version? Let me know if I can help you with anything.

Thank you and have a nice day.

[0] https://setuptools.readthedocs.io/en/latest/setuptools.html#metadata
[1] https://setuptools.readthedocs.io/en/latest/setuptools.html#new-and-changed-setup-keywords
[2] https://git.launchpad.net/pylibravatar/tree/setup.py

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers