[SSSD] [sssd PR#919][-Waiting for review] ci: improvements for on-demand builds

2019-11-13 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/919 Title: #919: ci: improvements for on-demand builds Label: -Waiting for review ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#919][+Accepted] ci: improvements for on-demand builds

2019-11-13 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/919 Title: #919: ci: improvements for on-demand builds Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#919][comment] ci: improvements for on-demand builds

2019-11-13 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/919 Title: #919: ci: improvements for on-demand builds alexey-tikhonov commented: """ ACK """ See the full comment at https://github.com/SSSD/sssd/pull/919#issuecomment-553555489 ___ sssd-devel mailing list --

[SSSD] [sssd PR#915][comment] Fix krb5-related debug

2019-11-13 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/915 Title: #915: Fix krb5-related debug sumit-bose commented: """ Hi @alexey-tikhonov, thanks for the patches. I had a comment about the 3rd one, please see inline. bye, Sumit """ See the full comment at

[SSSD] [sssd PR#938][comment] Don't use tabs for indentation

2019-11-13 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/938 Title: #938: Don't use tabs for indentation centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/938#issuecomment-553510508 ___

[SSSD] [sssd PR#938][opened] Don't use tabs for indentation

2019-11-13 Thread miztake
URL: https://github.com/SSSD/sssd/pull/938 Author: miztake Title: #938: Don't use tabs for indentation Action: opened PR body: """ The Coding guidelines include the following: https://docs.pagure.org/SSSD.sssd/developers/coding_style.html MUST: No tabs all indentation 4 spaces.

[SSSD] [sssd PR#937][opened] Unify the indentation to 4 spaces

2019-11-13 Thread miztake
URL: https://github.com/SSSD/sssd/pull/937 Author: miztake Title: #937: Unify the indentation to 4 spaces Action: opened PR body: """ The Coding guidelines include the following: https://docs.pagure.org/SSSD.sssd/developers/coding_style.html MUST: No tabs all indentation 4 spaces.

[SSSD] [sssd PR#937][comment] Unify the indentation to 4 spaces

2019-11-13 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/937 Title: #937: Unify the indentation to 4 spaces centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/937#issuecomment-553500886 ___

[SSSD] [sssd PR#905][comment] Don't ignore host entries in Group Policy security filters

2019-11-13 Thread abbra
URL: https://github.com/SSSD/sssd/pull/905 Title: #905: Don't ignore host entries in Group Policy security filters abbra commented: """ @dmulder they need to be reviewed by SSSD core team (I'm not the one ;)), so we need a review from @pbrezina and @sumit-bose, at least. """ See the full

[SSSD] [sssd PR#905][comment] Don't ignore host entries in Group Policy security filters

2019-11-13 Thread dmulder
URL: https://github.com/SSSD/sssd/pull/905 Title: #905: Don't ignore host entries in Group Policy security filters dmulder commented: """ @abbra Any thoughts on these latest patches? """ See the full comment at https://github.com/SSSD/sssd/pull/905#issuecomment-553475294

[SSSD] [sssd PR#933][comment] Unify the indentation to 4 spaces

2019-11-13 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/933 Title: #933: Unify the indentation to 4 spaces pbrezina commented: """ I'm fine with fixing the indentation, I just wonder if git can cope with different spaces when backporting patches? """ See the full comment at

[SSSD] [sssd PR#933][comment] Unify the indentation to 4 spaces

2019-11-13 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/933 Title: #933: Unify the indentation to 4 spaces pbrezina commented: """ I'm fine with fixing the indentation, I just wonder if git can cope with different backspaces when backporting patches? """ See the full comment at

[SSSD] [sssd PR#933][closed] Unify the indentation to 4 spaces

2019-11-13 Thread miztake
URL: https://github.com/SSSD/sssd/pull/933 Author: miztake Title: #933: Unify the indentation to 4 spaces Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/933/head:pr933 git checkout pr933

[SSSD] [sssd PR#933][comment] Unify the indentation to 4 spaces

2019-11-13 Thread miztake
URL: https://github.com/SSSD/sssd/pull/933 Title: #933: Unify the indentation to 4 spaces miztake commented: """ This PR alone cannot cover the 4 spaces indentation. I'm sorry, but I'll close this PR once and organize it again. # If you don't like many sources being modified by such PR, don't

[SSSD] [sssd PR#933][comment] Unify the indentation to 4 spaces

2019-11-13 Thread miztake
URL: https://github.com/SSSD/sssd/pull/933 Title: #933: Unify the indentation to 4 spaces miztake commented: """ This PR alone cannot cover the 4 spaces indentation. I'm sorry, but I'll close this PR once and organize it again. If you don't like many sources being modified by such PR, don't

[SSSD] [sssd PR#932][-Waiting for review] SBUS: defer deallocation of sbus_watch_ctx

2019-11-13 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/932 Title: #932: SBUS: defer deallocation of sbus_watch_ctx Label: -Waiting for review ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#934][+Waiting for review] SBUS: defer deallocation of sbus_watch_ctx

2019-11-13 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/934 Title: #934: SBUS: defer deallocation of sbus_watch_ctx Label: +Waiting for review ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to