Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conditional dependency to module 'enum34': Ony for python_version < "3.4.0" #16

Merged
merged 1 commit into from
Mar 10, 2021

Conversation

michael-koeller
Copy link
Contributor

The current master of rtrlib/python-binding may cause pip installations to fail on Python versions 3.6 - 3.8 because of the dependency to module enum34. This dependency is not needed for Python 3.4 and above.

This is a known issue with pip and seems to be fixed only recently (2020-12).

Since e.g. default Python version on Ubuntu 20.04 LTS is 3.8, where the issue will still be present, I think this project should guard against pip installation fails by adding a version contition to the dependency.

@michael-koeller michael-koeller changed the title Issue/enum34 Conditional dependency to module 'enum34' Mar 1, 2021
@michael-koeller michael-koeller changed the title Conditional dependency to module 'enum34' Conditional dependency to module 'enum34': Ony for python_version < "3.4.0" Mar 1, 2021
@mroethke
Copy link
Member

mroethke commented Mar 8, 2021

If I understand that bug report correctly this only occurs with a pyproject.toml, but we still use a setup.py.
And I never encountered this even though I develop on archlinux.
Probably makes sense to have this anyway.

Please rebase on master, than I will merge.

@michael-koeller
Copy link
Contributor Author

michael-koeller commented Mar 9, 2021 via email

@mroethke mroethke merged commit a4117ae into rtrlib:master Mar 10, 2021
@mroethke
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants