FS#60319 - [python-ifaddr] Missing dependency

Attached to Project: Community Packages
Opened by Sibren Vasse (SibrenVasse) - Friday, 05 October 2018, 16:43 GMT
Last edited by Jelle van der Waa (jelly) - Friday, 30 November 2018, 17:41 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Jelle van der Waa (jelly)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

Description:
python-ifaddr requires ipaddress.
https://github.com/pydron/ifaddr/blob/e6163693b6b6d85aad711c3a6849ce7e3bc80d01/ifaddr/_shared.py#L24

Additional info:
* python-ifaddr 0.1.4-1
* Example traceback in attachment
   log (1 KiB)
This task depends upon

Closed by  Jelle van der Waa (jelly)
Friday, 30 November 2018, 17:41 GMT
Reason for closing:  Fixed
Additional comments about closing:  Fixed in 0.1.5-1
Comment by Patrick Lühne (patrick.luehne) - Tuesday, 27 November 2018, 22:01 GMT
@SibrenVasse: I believe that the problem is not that the ipaddress dependency is missing in this PKGBUILD. Since Python 3.3, ipaddress is shipped as a core module [1], so it is, in fact, installed for every Python user on Arch Linux.

The actual problem is that python-ifaddr explicitly depends on the former (third-party, separately installed) ipaddress module, which isn’t available as such with Python 3.3 and newer (as it’s already contained in the vanilla Python installation). Hence, the dependency checker will fail to find ipaddress even though it’s implicitly available.

I think that the right solution is to restrict the ipaddress install dependency of ifaddr to Python earlier than 3.3 in the project’s setup.py. I proposed a pull request in the upstream repository [2], so let’s see what the maintainers say.

[1] https://docs.python.org/3/library/ipaddress.html
[2] https://github.com/pydron/ifaddr/pull/12
Comment by Patrick Lühne (patrick.luehne) - Wednesday, 28 November 2018, 02:42 GMT
The upstream maintainer merged my fix and released a patched version, python-ifaddr 0.1.5. This should resolve the seemingly missing dependency reported here :).
Comment by Sibren Vasse (SibrenVasse) - Wednesday, 28 November 2018, 13:19 GMT
That does seems to fix my problem. Thanks for diving into the issue! :)
Comment by Patrick Lühne (patrick.luehne) - Wednesday, 28 November 2018, 13:25 GMT
@SibrenVasse: Glad to hear that my patch worked for you as well :).

Loading...