On Fri, Jun 07, 2013 at 06:24:48PM +0200, Lukas Slebodnik wrote:
- I like idea of divided subpackages. If someone wants only ldap backend, he
needn't install samba-libs (and its dependencies)
- There isn't any rpmlint warnings.
I tested yum upgrade upgrade with installed sssd and freeipa-client. New packages were installed for dependencies: sssd-ad sssd-common sssd-ipa sssd-krb5 sssd-krb5-common sssd-ldap Everything worked as expected.
Then I decided to remove sssd-ad: yum remove sssd-ad and packeges "freeipa-client, sssd" were also removed. I was little bit confused, because I didn't want to remove sssd and sssd replied to getent command after packages "freeipa-client, sssd" were removed.
That's because freeipa-client currently requires sssd, we might want open a ticket to make them require just sssd-ipa.
I think, that other users may be also confused with this situation. Then I looked to the patch and I found out, that: --sssd is only "meta package",which require all backedns subpackages --sssd doesn't contain any useful files --everything important is in package sssd-common.
Maybe we should update package description of sssd and sssd-common. I hope that system administrators relies on output of "yum info" and there isn't it very well explained.
Thanks, I updated the summary of both sssd and sssd-common. Hopefully it would be clearer now.
Summary: Everything works well, but I was little bit confused.
Any other opinions?
One nitpick inline
+%package krb5-common +Summary: SSSD helpers needed for Kerberos and GSSAPI authentication +Group: Applications/System +License: GPLv3+ +Conflicts: sssd < %{version}-%{release} +Requires: cyrus-sasl-gssapi +Requires: sssd-common = %{version}-%{release} +Requires: sssd-krb5-common = %{version}-%{release}
^^^^^^^^^^^^^^^^Package sssd-krb5-common requires itself?
Sorry, this was meant to be placed only in the part with the sssd-krb5 subpackage, I think I copied the Requires to both. New patch is attached.