Bug 2208964 - Review Request: python-flit-core - PEP 517 build backend for packages using Flit
Summary: Review Request: python-flit-core - PEP 517 build backend for packages using Flit
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Maxwell G
QA Contact: Fedora Extras Quality Assurance
URL: https://flit.pypa.io/
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-05-22 08:19 UTC by Miro Hrončok
Modified: 2023-06-02 21:01 UTC (History)
2 users (show)

Fixed In Version: python-flit-core-3.8.0-3.fc39
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2023-06-02 21:01:36 UTC
Type: ---
Embargoed:
maxwell: fedora-review+


Attachments (Terms of Use)
The .spec file difference from Copr build 5941031 to 5942806 (885 bytes, patch)
2023-05-22 17:29 UTC, Fedora Review Service
no flags Details | Diff

Description Miro Hrončok 2023-05-22 08:19:38 UTC
Spec URL: https://src.fedoraproject.org/fork/churchyard/rpms/python-flit/raw/0620186a502085e2f7b833921297a9a4ddf1fb7b/f/python-flit-core.spec
SRPM URL: https://kojipkgs.fedoraproject.org//work/tasks/8668/101438668/python-flit-core-3.8.0-3.fc39.src.rpm

See also https://src.fedoraproject.org/rpms/python-flit/pull-request/28

Description:
This provides a PEP 517 build backend for packages using Flit.
The only public interface is the API specified by PEP 517,
at flit_core.buildapi.

Fedora Account System Username: churchyard

Comment 1 Fedora Review Service 2023-05-22 08:26:25 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/5941031
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2208964-python-flit-core/fedora-rawhide-x86_64/05941031-python-flit-core/fedora-review/review.txt

Please take a look if any issues were found.

---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 2 Miro Hrončok 2023-05-22 08:38:07 UTC
- Package does not use a name that already exists.
  Note: A package with this name already exists. Please check
  https://src.fedoraproject.org/rpms/python-flit-core
  See: https://docs.fedoraproject.org/en-US/packaging-
  guidelines/Naming/#_conflicting_package_names


Note that this is for unretirement. Technically, this now exists as EPEL-only package and I want to introduce it in Fedora.



> python3-flit-core.noarch: W: no-documentation

See the comment in the spec.

Comment 3 Maxwell G 2023-05-22 14:53:45 UTC
I suppose I'll continue the review in https://src.fedoraproject.org/rpms/python-flit/pull-request/28.

Comment 5 Fedora Review Service 2023-05-22 17:29:09 UTC
Created attachment 1966265 [details]
The .spec file difference from Copr build 5941031 to 5942806

Comment 6 Fedora Review Service 2023-05-22 17:29:11 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/5942806
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2208964-python-flit-core/fedora-rawhide-x86_64/05942806-python-flit-core/fedora-review/review.txt

Please take a look if any issues were found.

---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 7 Maxwell G 2023-05-22 18:37:06 UTC
- [x] The License tag reflects the package contents and uses the correct identifiers.

Specfile adjusted to account for flit_core/versionno.py

- [x] The license text is included and marked with %license.
- [x] The package builds successfully in mock.
- [x] The package is installable (checked by fedora-review).

- [x] There are no relevant rpmlint errors.

> python3-flit-core.noarch: W: no-documentation

is justified with a specfile comment.

- [x] The package runs tests in %check.

- [x] The latest version is packaged or packaging an earlier version is justified.

See https://src.fedoraproject.org/rpms/python-flit/pull-request/28#comment-143079.


- [x] The packager considers avoiding confusing `%foo_name` macros. (Not a blocker)
- [x] Libraries: The package name has a `python3-` prefix and uses the canonical project name
- [-] Applications: A `python3-` prefix is not used
- [x] The pyproject macros are used.
- [x] There are no bundled libraries.
- [x] The package complies with the Python and general Packaging Guidelines.


Package approved! Thanks for working on this! Tell releng that they can keep me as the main admin when you file the unretirement ticket. I added churchyard and @python-packagers-sig as admins.

Comment 8 Miro Hrončok 2023-05-22 19:17:05 UTC
https://pagure.io/releng/issue/11433


Note You need to log in before you can comment on or make changes to this bug.