URL: https://github.com/SSSD/sssd/pull/5685
Title: #5685: KCM: Add krb5-libs dependency in spec

rwmjones commented:
"""
I don't believe this change is correct.  `krb5-libs` was installed, it was just 
a too-old version, so this change wouldn't make any difference.  In fact RPM 
automatic dependencies already pick up the need for the library:

```
$ rpm -qR sssd-kcm | grep krb5
libkrb5.so.3()(64bit)
libkrb5.so.3(krb5_3_MIT)(64bit)
```

The real problem is in krb5-libs itself, where symbols aren't properly 
versioned.  Every time you add a new symbol (or more realisticly, when you do 
an upstream release containing new symbols) you should add a new version 
section to your linker script.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/5685#issuecomment-861434301
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

Reply via email to