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

[PEP 517] Relax installation-time "pep517<0.7.0" requirement #2573

Closed
leycec opened this issue Apr 5, 2022 · 1 comment · Fixed by #2680
Closed

[PEP 517] Relax installation-time "pep517<0.7.0" requirement #2573

leycec opened this issue Apr 5, 2022 · 1 comment · Fixed by #2680
Assignees

Comments

@leycec
Copy link

leycec commented Apr 5, 2022

python-for-android currently requires obsolete versions of the pep517 Python package at setuptools-based installation time via a pep517<0.7.0 requirement in the top-level setup.py script. This is bad for any number of obvious reasons, including:

  • pep517 0.6.0, the most recent version of this package to satisfy this constraint, is nearly three years old. In Python years, pep517 is the geriatric centenarian with insufferable incontinence that habitually screeches "Gituffmuhlaaaaawn!" while throwing mouldy wonder bread at passing codebases.
  • pep517 0.6.0 is so old that it's only tested to work under Python 3.4–3.7. Of those, only Python 3.7 has yet to hit its official End-of-Life (EOL) – but even that will happen shortly.
  • pep517 0.6.0 is no longer packaged by any system package managers (e.g., Debian's apt, Arch's pacman, Gentoo's emerge). It's unclear whether pep517 0.6.0 even can be packaged by any system package managers or run under actively developed Python versions. By transitivity, python-for-android itself can no longer be reasonably packaged for any Linux distros.

In short, the pep517<0.7.0 constraint needs to be relaxed to cover the most recent stable release of pep517. In theory, this should be trivial; the p4a codebase only internally imports from pep517 twice in the python-for-android.pythonpackage submodule.

Thanks as always for all the phenomenal volunteerism, p4a team! You make the impossible at least slightly possible. 🤠

@misl6 misl6 self-assigned this Apr 6, 2022
@misl6 misl6 added the Priority: Medium This issue may be useful, and needs some attention. label Apr 6, 2022
@misl6 misl6 linked a pull request Oct 5, 2022 that will close this issue
@misl6 misl6 removed the Priority: Medium This issue may be useful, and needs some attention. label Oct 6, 2022
@leycec
Copy link
Author

leycec commented Oct 6, 2022

Thanks so much, @misl6. Kivy devs the best devs. 🥰

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 a pull request may close this issue.

2 participants