#6875 Correct wheel package dependencies
Closed: fixed Opened by cheimes.

Python wheel packages have some incorrect dependencies:

  • Python 2.7 wheels use pyldap instead of python-ldap but Python 2.7 Fedora packages use python-ldap. FreeIPA should use the same library.
  • The extra dependency otptoken_yubikey specifies the wrong libraries. It's python-yubico and pysub, not yubico and usb. The names yubico and usb are the Python package names (e.g. import yubico), not the PyPI package names.
  • otptoken_yubikey should really be optional.
  • ipapython claims to depend on jwcrypto and requests but never imports any of the libraries directly.
  • On the other hand ipaserver depends on jwcrypto and requests but fails to declare the dependencies.
  • tox.ini references ipapython[certmonger] but this extra dependency does no longer exist. It was removed during development. pip only warns but not fails for missing extra dependencies.

All problems should be fixed in master and 4.5.


Metadata Update from @cheimes:
- Issue priority set to: blocker
- Issue set to the milestone: FreeIPA 4.5.1
- Issue tagged with: integration

Metadata Update from @pvoborni:
- Issue set to the milestone: 0.0 NEEDS_TRIAGE (was: FreeIPA 4.5.1)

Metadata Update from @pvoborni:
- Custom field rhbz adjusted to 0
- Issue priority set to: critical (was: blocker)
- Issue set to the milestone: FreeIPA 4.5.1 (was: 0.0 NEEDS_TRIAGE)

master:

  • 26ab51ddf47f421f3404709052db89f08c05adaa Correct PyPI package dependencies
  • 994d24d288080e924e039ca0a7b0b0dfc2355ac1 Band-aid for pip dependency bug

Not closing the ticket, 4.5.1 branch requires different patches

ipa-4-5:

  • b91ee1294bb3139f3d9df62c75dd429a5821bf40 Correct PyPI package dependencies

Metadata Update from @tkrizek:
- Issue close_status updated to: fixed
- Issue status updated to: Closed (was: Open)

Regression found. It is failing on RHEL.

https://github.com/freeipa/freeipa/pull/787

Metadata Update from @mbasti:
- Issue status updated to: Open (was: Closed)

master:

  • bea7236b9c5a82db5d945a88103c0524a793a8a2 ipasetup: fix dependencies handling based on python version

ipa-4-5:

  • c49e146a69a66cda894687f39f3d77ff3ad9c33b ipasetup: fix dependencies handling based on python version

Metadata Update from @mbasti:
- Issue close_status updated to: fixed
- Issue status updated to: Closed (was: Open)

Metadata