From 6fe5e182cc0a57e90a577c21e1e05c5b25bb8232 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Miro=20Hron=C4=8Dok?= <miro@hroncok.cz>
Date: Tue, 26 Sep 2023 15:13:08 +0200
Subject: [PATCH] Avoid needlesly BuildRequiring python3-rich when not running
 tests and on RHEL

---
 python-setuptools_scm.spec | 17 ++++++++++++++---
 1 file changed, 14 insertions(+), 3 deletions(-)

diff --git a/python-setuptools_scm.spec b/python-setuptools_scm.spec
index f09044d..556eb48 100644
--- a/python-setuptools_scm.spec
+++ b/python-setuptools_scm.spec
@@ -23,7 +23,7 @@ BuildRequires:  mercurial
 BuildRequires:  python3dist(pytest)
 BuildRequires:  python3dist(setuptools) >= 45
 # virtualenv omitted, see https://github.com/pypa/setuptools_scm/pull/940
-# rich omitted, pulled in as build-backend dependency
+# rich omitted, pulled in only with the [rich] extra
 %endif
 
 %description
@@ -39,15 +39,26 @@ Setuptools_scm handles managing your Python package versions in SCM metadata.
 It also handles file finders for the supported SCMs.
 
 
-%pyproject_extras_subpkg -n python%{python3_pkgversion}-setuptools_scm toml
+# We don't package the [rich] extra on RHELs, to avoid pulling rich into the buildroot
+%pyproject_extras_subpkg -n python%{python3_pkgversion}-setuptools_scm toml%{!?rhel:,rich}
 
 
 %prep
 %autosetup -p1 -n setuptools-scm-%{version}
+# Upstream bogusly declares rich as a build-system dependency,
+# but the build works without it.
+# We remove it here to simplify the bootstrap loop
+# and to avoid the dependency in RHEL builds.
+# Upstream issue: https://github.com/pypa/setuptools_scm/issues/941
+# Upstream PR: https://github.com/pypa/setuptools_scm/pull/942
+# The PR does not apply cleanly to 8.0.3, so we sed it out instead.
+# The sed removes the first line with "rich" only:
+sed -i '0,/"rich"/{/"rich"/d}' pyproject.toml
+
 
 
 %generate_buildrequires
-%pyproject_buildrequires
+%pyproject_buildrequires %{?with_tests:%{!?rhel:-x rich}}
 
 
 %build