[SSSD] [sssd PR#819][comment] BUILD: fix libpython handling in Python3.8

2019-05-24 Thread hroncok
URL: https://github.com/SSSD/sssd/pull/819 Title: #819: BUILD: fix libpython handling in Python3.8 hroncok commented: """ Thank you @sumit-bose Do you consider the name "embed" OK for your use case? If not, I can bring that to CPython upstream. """ See the full comment at https://github.com

[SSSD] [sssd PR#819][comment] BUILD: fix libpython handling in Python3.8

2019-05-24 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/819 Title: #819: BUILD: fix libpython handling in Python3.8 sumit-bose commented: """ Hi, if you want to test this with Python3.8, there is a repo for rawhide at https://copr.fedorainfracloud.org/coprs/g/python/python3.8/. bye, Sumit """ See the full co

[SSSD] [sssd PR#819][opened] BUILD: fix libpython handling in Python3.8

2019-05-24 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/819 Author: sumit-bose Title: #819: BUILD: fix libpython handling in Python3.8 Action: opened PR body: """ Related to https://pagure.io/SSSD/sssd/issue/4016 """ To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch gh

[SSSD] [sssd PR#818][comment] DP: add NULL check to be_ptask_{enable|disable}

2019-05-24 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/818 Title: #818: DP: add NULL check to be_ptask_{enable|disable} alexey-tikhonov commented: """ Honestly, I would prefer ``` if (!task) { return; } ``` instead of wrapping whole function content into `if (task != NULL) {}` At least `diff` would be smal

[SSSD] [sssd PR#818][comment] DP: add NULL check to be_ptask_{enable|disable}

2019-05-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/818 Title: #818: DP: add NULL check to be_ptask_{enable|disable} jhrozek commented: """ @thalman Hi Tomas, sorry I didn't clone and assign the bug to you earlier, but I wonder if you can look at the patch and review it? """ See the full comment at https

[SSSD] [sssd PR#818][opened] DP: add NULL check to be_ptask_{enable|disable}

2019-05-24 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/818 Author: sumit-bose Title: #818: DP: add NULL check to be_ptask_{enable|disable} Action: opened PR body: """ Currently the files and the proxy provider do not provide a check online method (DPM_CHECK_ONLINE). The files provider because it can never go

[SSSD] [sssd PR#817][synchronized] LDAP: Return the error message from the extdom password change also on failure

2019-05-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/817 Author: jhrozek Title: #817: LDAP: Return the error message from the extdom password change also on failure Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/817/head:pr817 git

[SSSD] [sssd PR#817][comment] LDAP: Return the error message from the extdom password change also on failure

2019-05-24 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/817 Title: #817: LDAP: Return the error message from the extdom password change also on failure jhrozek commented: """ thank you for the review, I fixed the commit message """ See the full comment at https://github.com/SSSD/sssd/pull/817#issuecomment-495

[SSSD] [sssd PR#814][comment] certmap: allow missing KU in OpenSSL version

2019-05-24 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/814 Title: #814: certmap: allow missing KU in OpenSSL version sumit-bose commented: """ Hi, the lastest version should fix both comments. bye, Sumit """ See the full comment at https://github.com/SSSD/sssd/pull/814#issuecomment-495509912 ___

[SSSD] [sssd PR#814][synchronized] certmap: allow missing KU in OpenSSL version

2019-05-24 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/814 Author: sumit-bose Title: #814: certmap: allow missing KU in OpenSSL version Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/814/head:pr814 git checkout pr814 From 07173200b9f

[SSSD] [sssd PR#817][comment] LDAP: Return the error message from the extdom password change also on failure

2019-05-24 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/817 Title: #817: LDAP: Return the error message from the extdom password change also on failure pbrezina commented: """ Ack. """ See the full comment at https://github.com/SSSD/sssd/pull/817#issuecomment-495497013

[SSSD] [sssd PR#817][+Accepted] LDAP: Return the error message from the extdom password change also on failure

2019-05-24 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/817 Title: #817: LDAP: Return the error message from the extdom password change also on failure Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to ss