Bug 2347199

Summary: Please branch and build pre-commit in epel10
Product: [Fedora] Fedora EPEL Reporter: Romain Geissler <romain.geissler>
Component: pre-commitAssignee: Ben Beasley <code>
Status: CLOSED ERRATA QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: epel10CC: aniketpradhan1999, chedi.toueiti, code, python-packagers-sig
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: pre-commit-4.1.0-6.el10_1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2025-03-07 04:00:48 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 2347197    
Bug Blocks:    

Description Romain Geissler 2025-02-23 18:29:43 UTC
Hi,

Please branch and build pre-commit in epel10.

Comment 1 Ben Beasley 2025-02-23 23:49:50 UTC
I don’t necessarily object to branching pre-commit in EPEL10, if all of its dependencies end up being available there at some point, but it’s not needed for python-ipykernel. It is brought in by the test extra, where it would only be used for linting tests that we don’t (and shouldn’t[1]) run. It can be easily removed.[2]

[1] https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_linters
[2] https://src.fedoraproject.org/rpms/python-ipykernel/pull-request/28

Comment 2 Ben Beasley 2025-02-23 23:56:41 UTC
Based on local mock builds, python-ukkonen and python-identify really are the last missing dependencies in EPEL10, and pre-commit would build successfully if those two packages were branched and built.

Comment 3 Fedora Admin user for bugzilla script actions 2025-02-24 13:55:34 UTC
This package has changed maintainer in Fedora. Reassigning to the new maintainer of this component.

Comment 4 Romain Geissler 2025-02-26 15:36:18 UTC
This package indeed isn't needed anymore to build anything else. It can however be built now as all dependencies are now available: https://koji.fedoraproject.org/koji/taskinfo?taskID=129631500

Comment 5 Ben Beasley 2025-02-26 16:18:18 UTC
(In reply to Romain Geissler from comment #4)
> This package indeed isn't needed anymore to build anything else. It can
> however be built now as all dependencies are now available:
> https://koji.fedoraproject.org/koji/taskinfo?taskID=129631500

The dependencies were built for epel10 (10.1) but not for epel10.0, so I’m going to follow suit with pre-commit. If someone has a particular need for epel10.0, speak up: I’m sure that python-identify/python-ukkonen could be branched there as well.

Comment 6 Fedora Update System 2025-02-26 16:42:13 UTC
FEDORA-EPEL-2025-f489f96798 (pre-commit-4.1.0-6.el10_1) has been submitted as an update to Fedora EPEL 10.1.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2025-f489f96798

Comment 7 Fedora Update System 2025-02-27 02:59:10 UTC
FEDORA-EPEL-2025-f489f96798 has been pushed to the Fedora EPEL 10.1 testing repository.

You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2025-f489f96798

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 8 Fedora Update System 2025-03-07 04:00:48 UTC
FEDORA-EPEL-2025-f489f96798 (pre-commit-4.1.0-6.el10_1) has been pushed to the Fedora EPEL 10.1 stable repository.
If problem still persists, please make note of it in this bug report.