Churchyard (talk | contribs) |
Churchyard (talk | contribs) |
||
| Line 131: | Line 131: | ||
Python 4 is not planned upstream. | Python 4 is not planned upstream. | ||
==== Tests and | ==== Tests, linters and coverage ==== | ||
Running upstream tests is mandatory. | Running upstream tests is mandatory. | ||
Running linters (e.g. {{package|pylint}}, {{package|pyflakes}}, | Running linters (e.g. {{package|pylint}}, {{package|pyflakes}}, | ||
{{package|black}}) is discouraged. | {{package|python-black}}) | ||
or test coverage measurements {{package|python-coverage}} is discouraged. | |||
=== Non-Guideline Changes === | === Non-Guideline Changes === | ||
Revision as of 12:09, 11 June 2021
Python Packaging Guidelines overhaul
Summary
The Python Packaging guidelines will be rewritten, with the major changes being PyPI parity and usage of upstream metadata.
A new set of macros, 
pyproject-rpm-macros
The older (a.k.a. "201x-era") Python Packaging guidelines will remain in effect as an option (until retired by another Fedora Change).
Owner
- Name: Petr Viktorin
- Email: pviktori@redhat.com
- Name: Python SIG
- Email: python-devel@lists.fedoraproject.org
Current status
- Targeted release: Fedora Linux 35
- Last updated: 2021-06-11
- FESCo issue: <will be assigned by the Wrangler>
- Tracker bug: <will be assigned by the Wrangler>
- Release notes tracker: <will be assigned by the Wrangler>
Detailed Description
New Python packaging guidelines are available at this Pull request: XXX https://pagure.io/fork/pviktori/packaging-committee/diff/master..new-py-guidelines
A rendered version (which might lag behind the pull request) is at: XXX
Not removing older guidelines
The current ("201x-era") guidelines will stay in effect as an option for packages that haven't migrated yet or those that cannot follow the new guidelines for whatever unforeseen reason.
They will be retired in another Fedora Change, some time after the vast majority of Python packages follow the new guidelines and there are no known blockers for the remaining ones. There's no rush; it might well take a decade.
Guideline Changes
See an external document for a detailed list of changes to MUST/SHOULD rules. The major ones are listed here:
Requiring python3-devel
All packages that use Python at run- or build-time will need to
BuildRequire
PyPI Parity
Machine-readable metadata (distribution names in
dist-info directories on disk and the corresponding
python3.Xdist(foo) RPM provides) will match the Python
Package Index (PyPI).
This solves a namespace issue. Python packaging tools use a flat namespace,
and PyPI is the place where open-source Python packages are generally
published, so users and tools assume a package called requests
is whatever requests means on PyPI.
While this is not ideal (especially for private packages), it makes sense
for Fedora to align with the PyPI namespace.
Note that Fedora package names are not affected – just the Python packaging metadata on disk and virtual RPM Provides generated from it.
The new guidelines cover what to do for packages that cannot be registered on PyPI. The Change owner is prepared to help with PyPI registration.
Note that names found in Fedora but not on PyPI have been reserved on PyPI to avoid being taken by unrelated projects.
Upstream metadata
Upstream “dist-info” metadata for Python packages is now standardized and
shareable with other distributors (be it Linux distros or others).
As much as possible, RPM metadata should be automatically taken from there,
rather be duplicated in the spec file (and likely diverge over time).
This includes run-time and build-time requirements for Python packages, tests
and test requirements for packages that use the tox tool,
and Extras (optional features).
Packagers are expected to treat metadata bugs as any other bugs. (That is, ideally: patch them and present the patches upstream).
Naming
The new guidelines explain the various names a Python package can have (Fedora package name, project name, import name) and begs developers/maintainers to keep them synchronized if possible.
Python 3
The new guidelines only cover Python 3.
Python 2 does not need its own guidelines.
Python 4 is not planned upstream.
Tests, linters and coverage
Running upstream tests is mandatory.
Running linters (e.g.
Non-Guideline Changes
The
While this small package is not always necessary, it will make life easier for most Python packagers.
Feedback
The new guidelines wede discussed extensively on the Python SIG list discussion thread.
Two major objections remain. Both were best voiced by Neal Gompa:
Burdening packagers
Neal does “not necessarily disagree that PyPI and Fedora pythonXdist() names should match, but [he disagrees] on burdening packagers with this.”
However, actually pushing a package to PyPI (and maintaining it there) is not necessary. The minimal-effort solution is to reserve the name on PyPI so that a conflicting package can not be created there.
The author of this change is willing to do this work for all Python packages whose upstream is not on PyPI; packagers need to send a request to the Python SIG list. If something happens to me, Red Hat's python-maint team is also ready to do this, and in the event of catastrophic management change, PyPI maintainers can be e-mailed directly.
This is all described in the new guidelines (see the PyPI parity section).
The new guidelines discourage the minimal-effort solution because it is suboptimal for users (and for the wider Python ecosystem).
(It was further suggested that the registration process should be automatic to minimize burden on packagers. It will be automated if it's too much for humans to handle, but we expect there won't be enough requests to make automation worth it.)
Conditioning names on PyPI
Neal also disagrees “on conditioning those names on PyPI, because frankly no other ecosystem in Fedora works that way. Not Rust, not Go, not Perl, not OCaml, not anything.”
It's true that no other ecosystem in Fedora does this, but the author of this change believes that they should: if the ecosystem uses a flat namespace of dependencies, its Fedora packages should use the ecosystem-wide namespace rather than a Fedora-specific one. RPM predates global per-ecosystem directories of free/open software, but they are here and users (at least those affected by these issues) are used to them.
For Python specifically, PyPI is already a de-facto standard:
if you see a name in install_requires in setup.py
of some random upstream project you want to package,
you can assume that it's the PyPI name.
It is in the best interest of Fedora's users to use PyPI names for dependency resolution.
Benefit to Fedora
The new packaging guidelines should encourage sharing packaging work with upstream developers (and through them, with other distros). Hopefully, they'll also free up some packagers' time.
PyPI parity will hopefully enable automated ways of mixing Fedora packages with upstream ones.
The updated examples and notes will be more useful to today's packagers.
Scope
- Policies and guidelines: XXX https://pagure.io/fork/pviktori/packaging-committee/diff/master..new-py-guidelines
- Proposal owners:
- Make
python3-develpyproject-rpm-macros - Remove provisional status from
- Handle PyPI name registration/reservation requests from other packagers.
- Make
- Other developers: Maintainers of packages that use Python should gradually switch to the new guidelines
- Release engineering: XXX #Releng issue number
- Trademark approval: N/A (not needed for this Change)
- Alignment with Objectives: N/A
Upgrade/compatibility impact
None expected. Any compatibility impact on individual packages would be a bug.
How To Test
If you maintain a package that uses Python, try to follow the new guidelines. Any major omission or ambiguity is a bug. (Minor/unique cases should be discussed on the Pytohn SIG list instead; the document is already quite long.)
Updated packages should be tested normally, as with any other package update.
User Experience
No direct impact on end users.
Dependencies
PyPI maintainers are aware of the general direction the Guidelines, and have been supportive.
Contingency Plan
- Contingency mechanism: Revert to the older guidelines
- Contingency deadline: N/A (Any time until old guidelines are retired, in a separate Change)
- Blocks release? No
Documentation
The new guidelines are the documentation. See Detailed Description above for links.
Release Notes
Fedora's Python Packaging guidelines have been rewritten to align with modern Python packaging practices and better integrate with the wider Python ecosystem (specifically, the Python Package Index).