PR#18: Limit python3-libsss_nss_idmap build dependency to server arches - rpms/freeipa

rpms / freeipa

#18 Limit python3-libsss_nss_idmap build dependency to server arches

Closed 3 months ago by yselkowitz. Opened 3 months ago by yselkowitz.

rpms/ yselkowitz/freeipa rawhide into rawhide

Limit python3-libsss_nss_idmap build dependency to server arches

Yaakov Selkowitz • 3 months ago

153bea1

freeipa.spec

file modified

+1 -1

		`@@ -294,6 +294,7 @@`
		`BuildRequires: libsss_idmap-devel`
		`BuildRequires: libsss_certmap-devel`
		`BuildRequires: libsss_nss_idmap-devel >= %{sssd_version}`
		`+ BuildRequires: python3-libsss_nss_idmap`
		`%if 0%{?fedora} >= 39 \|\| 0%{?rhel} >= 10`
		`# Do not use nodejs20 on fedora < 39, https://pagure.io/freeipa/issue/9374`
		`BuildRequires: nodejs(abi)`
		`@@ -321,7 +322,6 @@`
		`BuildRequires: python3-cffi`
		`BuildRequires: python3-dns`
		`BuildRequires: python3-ldap >= %{python_ldap_version}`
		`- BuildRequires: python3-libsss_nss_idmap`
		`BuildRequires: python3-netaddr >= %{python_netaddr_version}`
		`BuildRequires: python3-pyasn1`
		`BuildRequires: python3-pyasn1-modules`

yselkowitz commented 3 months ago

sssd was just disabled on i686 in ELN, and all references to pysss_nss_idmap are in ipaserver.

rcritten commented 3 months ago

The change looks good. I suppose I'd prefer this also be submitted upstream but it's not a big deal. I don't see how this is related to the SSSD i686 change though.

yselkowitz commented 3 months ago

python3-libsss_nss_idmap is a subpackage of sssd, so it was not available for the i686 build. However, sssd has since been re-enabled for i686 to avoid further dependency breakage, so it seems this is moot.

Pull-Request has been closed by yselkowitz

3 months ago

rcritten commented 3 months ago

I think this is a valid change. There is no need for this package when doing a client-only build. I'll submit this change upstream.