URL: https://github.com/SSSD/sssd/pull/720
Title: #720: contrib/ci/deps.sh: added missing dependency
alexey-tikhonov commented:
"""
Created new issue https://pagure.io/SSSD/sssd/issue/3914
Closing this PR.
"""
See the full comment at
URL: https://github.com/SSSD/sssd/pull/720
Title: #720: contrib/ci/deps.sh: added missing dependency
alexey-tikhonov commented:
"""
Thanks for explanations!
Does it mean that if test_pam_responder.py will be fixed properly then
libnss3-dev is not needed as well in Debian dependency list?
"""
URL: https://github.com/SSSD/sssd/pull/720
Title: #720: contrib/ci/deps.sh: added missing dependency
sumit-bose commented:
"""
I'm sorry, I think should have given more context. SSSD can be build with
either NSS or OpenSSL crypto libraries.
The CI build for Debian is using OpenSSL which can
URL: https://github.com/SSSD/sssd/pull/720
Title: #720: contrib/ci/deps.sh: added missing dependency
alexey-tikhonov commented:
"""
> Adding it as a dependency is ok, however if it is possible with pytest to run
> the related fixtures only for the NSS build this would be even better.
I agree
URL: https://github.com/SSSD/sssd/pull/720
Title: #720: contrib/ci/deps.sh: added missing dependency
mzidek-rh commented:
"""
> @alexey-tikhonov is right that test_pam_responder.py uses certutil
> unconditionally.
>
> Adding it as a dependency is ok, however if it is possible with pytest to
URL: https://github.com/SSSD/sssd/pull/720
Title: #720: contrib/ci/deps.sh: added missing dependency
sumit-bose commented:
"""
@alexey-tikhonov is right that test_pam_responder.py uses certutil
unconditionally.
Adding it as a dependency is ok, however if it is possible with pytest to run
the
URL: https://github.com/SSSD/sssd/pull/720
Title: #720: contrib/ci/deps.sh: added missing dependency
jhrozek commented:
"""
@sumit-bose do you have an opinion?
"""
See the full comment at
https://github.com/SSSD/sssd/pull/720#issuecomment-448606952