#1072 Python Packaging Guidelines overhaul
Merged by churchyard. Opened by pviktori.
pviktori/packaging-committee new-py-guidelines  into  master

Download 1072.patch

Here is a proposal for new Python packaging guidelines.
See the Fedora change at: https://fedoraproject.org/wiki/Changes/PythonPackagingGuidelines202x

Thank you for your contribution. Use the following link to see a preview of your contribution.
100.26.217.43/packaging-committee-pr1072. Do keep in mind that the build gets deleted if there is no update for more than a period of 2 weeks.

That is: http://100.26.217.43/packaging-committee-pr1072/packaging-guidelines/Python/

1 new commit added

  • Add missing space to fix the formatting

1 new commit added

  • Be consistent in license spelling

1 new commit added

  • Be consistent with %py_provides and %py_byte_compile usage

1 new commit added

  • More accurate explanation of %__pythonname_provides

1 new commit added

  • %python_provides never happened

2 new commits added

  • Don't use %py3_dist in the BR: pytest example
  • Quote 'requests[security]' in a command; [] are special to the shell

1 new commit added

  • Running tests: MUST → SHOULD

The Change was approved now.
But I'd still like to add documentation on %py3_check_import, which should hopefully be ready in a week or two: https://lists.fedoraproject.org/archives/list/packaging@lists.fedoraproject.org/thread/C4AHZ7EIQUM2YXEF6HU3BCNZUIIQ22EW/

Metadata Update from @james:
- Pull-request tagged with: meeting

For reference, there is also a human-created diff linked from the change proposal: https://hackmd.io/@python-maint/rJmQQc4DP

But I'd still like to add documentation on %py3_check_import, which should hopefully be ready in a week or two: https://lists.fedoraproject.org/archives/list/packaging@lists.fedoraproject.org/thread/C4AHZ7EIQUM2YXEF6HU3BCNZUIIQ22EW/

Available in Rawhide now. I've added a blurb to the current guidelines in https://pagure.io/packaging-committee/pull-request/1081 -- ready to be expanded in the test section of the new ones.

We discussed this at this weeks meeting (https://meetbot-raw.fedoraproject.org/fedora-meeting-1/2021-07-08/fpc.2021-07-08-16.08.txt):

  • #pr-1072 Python Packaging Guidelines overhaul (geppetto, 16:27:04)
  • LINK: https://pagure.io/packaging-committee/pull-request/1072
    (geppetto, 16:27:10)
  • ACTION: Python Packaging Guidelines overhaul (+1:6, 0:0, -1:0)
    (geppetto, 16:40:48)
  • Don't merge until Petr cleans the commits and adds %py3_check_import
    info (geppetto, 16:46:54)

(Packages using these usually use the <+%py3_install+>> or <+%py3_install_wheel+ macro>>...

The macro links don't seem to work in this case because the anchors are not present in this document.

rebased onto 418db93021d77a5a1cc8b52f1f67869395b20d70

I've rebased (to solve a conflict), and added a few new changes as commits for easier review.
It's ready to merge from my POV.

This is ready to be merged. I plan to do it once python3-devel requires pyproject-rpm-macros.

rebased onto f5723f9e0ba645eb417ad77aef1386cb1c5dff05

Pull-Request has been merged by churchyard

Metadata