URL: https://github.com/SSSD/sssd/pull/5910
Title: #5910: SPEC: avoid weak dependencies

sumit-bose commented:
"""
> > Please see my in-line comments.
> 
> Thank you. I updated the patch.
> 
> But what do you think about:
> 
> > I'm not sure what to do with:
> > ```
> > Recommends: bind-utils
> > Recommends: adcli
> > ```
> > 
> > 
> >     
> >       
> >     
> > 
> >       
> >     
> > 
> >     
> >   
> > bind-utils provides nsupdate used by IPA and AD backends, but SSSD can work 
> > without it. And adcli is used by AD backend to renew machine password.
> 
> ?

Hi,

since the related features are enabled in SSSD by default it would be good if 
the two packages will be installed together with `sssd-ad` or `sssd-ipa`. But 
since both features can be disabled a user might want to removed those packages 
if not needed. So `Recommends` would be best.

If I understood the original bugzilla ticket correctly we can keep 
`Recommends:` because neither `sssd-ad` nor `sssd-ipa` are in the `core` group 
(only `sssd-common` and `sssd-kcm` are).

HTH

bye,
Sumit


"""

See the full comment at 
https://github.com/SSSD/sssd/pull/5910#issuecomment-992344439
_______________________________________________
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