PyPI package should depend on pydns

Bug #1661218 reported by Peter Mattern
This bug affects 1 person
Affects Status Importance Assigned to Milestone

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.

Revision history for this message
François Marier (fmarier) wrote :

The dependency is in there:

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

Changed in pylibravatar:
status: New → Incomplete
Revision history for this message
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.

Revision history for this message
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
Revision history for this message
François Marier (fmarier) wrote :
Revision history for this message
François Marier (fmarier) wrote :

Another article on the difference between and requirements.txt:

Revision history for this message
frenzy (frenzy-madness) wrote :


I also have some troubles because of this bug.

Actually, it doesn't matter if you define your dependencies in 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 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 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.


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

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.