#940 Revisit the Python guidelines
Merged by james. Opened by churchyard.
churchyard/packaging-committee python  into  master

Download 940.patch
  • drop references to EOL Fedoras
  • introduce %python3, %python2 (and mention %python)
  • introduce %pycached
  • introduce %python3_platform
  • drop _python_bytecompile_extra - we will work to deprecate it

Full %pycached specification is hidden in
https://src.fedoraproject.org/rpms/python-rpm-macros/c/6c63a5b7f484f3c4b6bee13196561c9b1dff410a
But I deemed it too overwhelming to be put into the guidelines.

This is blocked by:

  • https://src.fedoraproject.org/rpms/python-rpm-macros/pull-request/38
  • https://src.fedoraproject.org/rpms/python-rpm-macros/pull-request/39
  • https://src.fedoraproject.org/rpms/epel-rpm-macros/pull-request/13
  • https://src.fedoraproject.org/rpms/epel-rpm-macros/pull-request/14
  • https://src.fedoraproject.org/rpms/python-rpm-macros/pull-request/40

I would drop first sentence. The guidelines are for Fedora only, so I don't think this makes much sense to keep here.

I always recommend putting this to the top of the spec file.

Let's use NOTE: … so that it becomes a separate block.

That would leave: "This generator is enabled by default." which I think may as well stay as written.

This line was just moved.

rebased onto eb7d888cf0b4e059d290854e0b0bc65eec74edf6

Done.

Who is "we"? Consider e.g. "it is usually placed just before"...

without a FESCo exception

FPC? This sentence was just moved around, but I can change it as well.

rebased onto 887f0341f31c92bb32b06ba545b95d3d8fb0e693

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

We discussed this at this weeks meeting (https://meetbot-raw.fedoraproject.org/fedora-meeting-1/2020-02-20/fpc.2020-02-20-17.10.txt):

  • #pr-940 Revisit the Python guidelines (geppetto, 17:22:35)
  • LINK: https://pagure.io/packaging-committee/pull-request/940
    (geppetto, 17:22:42)
  • ACTION: Revisit the Python guidelines (+1:5, 0:0, -1:0) (geppetto,
    17:25:50)

Commit dff2d50b fixes this pull-request

Pull-Request has been merged by james

Pull-Request has been merged by james

Metadata