[SSSD] [PATCH] RPM: Add new subpackage for PAC responder

Jakub Hrozek jhrozek at redhat.com
Wed Sep 4 16:31:58 UTC 2013


On Wed, Sep 04, 2013 at 11:15:51AM -0400, Stephen Gallagher wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> It was discovered that duplicating files in two subpackages is not
> permitted by Fedora packaging guidelines[1]. This patch moves the PAC
> responder to a new sssd-common-pac subpackage that both the sssd-ipa
> and sssd-ad subpackages will require.
> 
> [1]
> https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#DuplicateFiles
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.14 (GNU/Linux)
> Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
> 
> iEYEARECAAYFAlInTqcACgkQeiVVYja6o6NCywCeJTP7WIHDPfFkGActAtl/D0XB
> qQgAoJDmw93DuqCkHqELCInFSHJXZXHv
> =wk7f
> -----END PGP SIGNATURE-----

> From b9a5c02af52b042d00c723d70e698deb81fd763c Mon Sep 17 00:00:00 2001
> From: Stephen Gallagher <sgallagh at redhat.com>
> Date: Wed, 4 Sep 2013 11:12:30 -0400
> Subject: [PATCH] RPM: Add new subpackage for PAC responder
> 
> It was discovered that duplicating files in two subpackages is not
> permitted by Fedora packaging guidelines[1]. This patch moves the PAC
> responder to a new sssd-common-pac subpackage that both the sssd-ipa
> and sssd-ad subpackages will require.
> 
> [1] https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#DuplicateFiles
> ---
>  contrib/sssd.spec.in | 39 +++++++++++++++++++++++++++++++--------
>  1 file changed, 31 insertions(+), 8 deletions(-)
> 
> diff --git a/contrib/sssd.spec.in b/contrib/sssd.spec.in
> index 217625a3a0e551f5a99e498773ff71dfea1a32ed..71c400b13e525dfda8ffa22a26e2922420dfe1d9 100644
> --- a/contrib/sssd.spec.in
> +++ b/contrib/sssd.spec.in
> @@ -59,6 +59,7 @@ Requires: sssd-common = %{version}-%{release}
>  Requires: sssd-ldap = %{version}-%{release}
>  Requires: sssd-krb5 = %{version}-%{release}
>  Requires: sssd-ipa = %{version}-%{release}
> +Requires: sssd-common-pac = %{version}-%{release}
>  Requires: sssd-ad = %{version}-%{release}
>  Requires: sssd-proxy = %{version}-%{release}
>  Requires: python-sssdconfig = %{version}-%{release}
> @@ -243,6 +244,19 @@ Requires: sssd-krb5-common = %{version}-%{release}
>  Provides the Kerberos back end that the SSSD can utilize authenticate
>  against a Kerberos server.
>  
> +# RHEL 5 is too old to support the PAC responder
> +%if !0%{?is_rhel5}
> +%package common-pac
> +Summary: Common files needed for supporting PAC processing
> +Group: Applications/System
> +License: GPLv3+
> +Conflicts: sssd-ad < %{version}-{release}

Shouldn't this new subpackage alco Conflict with sssd-ipa ?

> +
> +%description common-pac
> +Provides common files needed by SSSD providers such as IPA and Active Directory
> +for handling Kerberos PACs.
> +%endif #is_rhel5

Othewise looks good and builds fine.



More information about the sssd-devel mailing list