PR#10: Fix pyproject.toml metadata - rpms/python-rapidfuzz

rpms / python-rapidfuzz

#10 Fix pyproject.toml metadata

Merged a year ago by troycurtisjr. Opened a year ago by lecris.

rpms/ lecris/python-rapidfuzz fix/python-metadata into rawhide

Cristian Le • a year ago

9ca47d3

433.patch

file added

+22

		`@@ -0,0 +1,22 @@`
		`+ From 2168b469b5b27be305c62d7f6c0c7b3df3d75f8f Mon Sep 17 00:00:00 2001`
		`+ From: Cristian Le <[email protected]>`
		`+ Date: Fri, 28 Feb 2025 13:06:24 +0100`
		`+ Subject: [PATCH] fix: pyproject.toml metadata`
		`+`
		`+ ---`
		`+ pyproject.toml \| 2 ++`
		`+ 1 file changed, 2 insertions(+)`
		`+`
		`+ diff --git a/pyproject.toml b/pyproject.toml`
		`+ index 3ffbf330..46b22a69 100644`
		`+ --- a/pyproject.toml`
		`+ +++ b/pyproject.toml`
		`+ @@ -23,6 +23,8 @@ classifiers=[`
		`+ "Programming Language :: Python :: 3.13",`
		`+ "License :: OSI Approved :: MIT License",`
		`+ ]`
		`+ +`
		`+ +[project.urls]`
		`+ Homepage = "https://github.com/rapidfuzz/RapidFuzz"`
		`+ Documentation = "https://rapidfuzz.github.io/RapidFuzz/"`
		`+ Repository = "https://github.com/rapidfuzz/RapidFuzz.git"`

python-rapidfuzz.spec

file modified

		`@@ -7,6 +7,8 @@`
		`URL: https://github.com/maxbachmann/RapidFuzz`
		`Source: %{pypi_source rapidfuzz}`

		`+ Patch: https://github.com/rapidfuzz/RapidFuzz/pull/433.patch`
		`+`
		`BuildRequires: gcc`
		`BuildRequires: gcc-c++`
		`BuildRequires: python3-devel`

lecris commented a year ago

Caught this issue when updating scikit-build-core 0.11

Blocking: https://src.fedoraproject.org/rpms/python-scikit-build-core/pull-request/74
Copr-build: https://copr.fedorainfracloud.org/coprs/lecris/scikit-build-core-0.11/builds

zuul commented a year ago

Build succeeded.
https://fedora.softwarefactory-project.io/zuul/buildset/c180f9a55fed412898e4914e0e258cde

check-for-arches : SUCCESS in 2m 44s (non-voting)
check-for-eln : SUCCESS in 17s
eln-rpm-scratch-build : SUCCESS in 15m 55s
rpm-scratch-build : SUCCESS in 16m 16s
rpm-scratch-build-s390x : SUCCESS in 12m 41s (non-voting)
rpm-scratch-build-ppc64le : SUCCESS in 13m 27s (non-voting)
rpm-scratch-build-i686 : SUCCESS in 16m 20s (non-voting)
rpm-scratch-build-aarch64 : SUCCESS in 13m 51s (non-voting)
rpm-linter : FAILURE in 4m 21s (non-voting)
rpm-rpminspect : SUCCESS in 9m 06s (non-voting)
check-for-sti-tests : SUCCESS in 14s
check-for-fmf-tests : SUCCESS in 14s
rpm-install-test : SUCCESS in 1m 39s
Skipped 2 jobs

troycurtisjr commented a year ago

Since this is changes to the upstream code and isn't Fedora specific, there should be an upstream PR linked to this patch as well.

lecris commented a year ago

You mean in the changelog? Otherwise the link to the upstream PRis in the Patch url itself

https://github.com/rapidfuzz/RapidFuzz/pull/433.patch
(Delete the .patch)

troycurtisjr commented a year ago

Sorry I was so expecting it as a comment, I didn't notice the patch URL itself was pointing upstream

Pull-Request has been merged by troycurtisjr

a year ago

lecris commented a year ago

Thanks, can you merge it for F42 also? Here are the side-tags you can use f43-build-side-106821, f42-build-side-106823 (although not crucial if you already submitted)