[SSSD] [sssd PR#488][comment] Two fixes for certificates in idoverrides

2018-02-27 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/488 Title: #488: Two fixes for certificates in idoverrides fidencio commented: """ Just for the record: backported patches do look fine by me and I do believe just having those 4 patches backported make sense. I still would like to know whether it

[SSSD] [sssd PR#519][comment] DEBUG: Print simple access provider allow and deny lists

2018-02-27 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/519 Title: #519: DEBUG: Print simple access provider allow and deny lists justin-stephenson commented: """ PR updated with changes suggested, please review. """ See the full comment at https://github.com/SSSD/sssd/pull/519#issuecomment-368991183

[SSSD] [sssd PR#519][synchronized] DEBUG: Print simple access provider allow and deny lists

2018-02-27 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/519 Author: justin-stephenson Title: #519: DEBUG: Print simple access provider allow and deny lists Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/519/head:pr519 git checkout

[SSSD] [sssd PR#515][comment] sssctl: Showing help even when sssd not configured

2018-02-27 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/515 Title: #515: sssctl: Showing help even when sssd not configured pbrezina commented: """ No, this is not correct usage of popt. You want something like this: ```c static void sss_tool_common_opts(struct sss_tool_ctx *tool_ctx, -

[SSSD] [sssd PR#488][comment] Two fixes for certificates in idoverrides

2018-02-27 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/488 Title: #488: Two fixes for certificates in idoverrides jhrozek commented: """ Yes, but one of the ssh patches was a dependency for 4300385 so I thought it's safer to just cleanly apply patches and have one less bug in sssd-1-13 at the same time. Of

[SSSD] [sssd PR#523][+Accepted] IPA: Drop unused ifdef HAVE_SELINUX_LOGIN_DIR

2018-02-27 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/523 Title: #523: IPA: Drop unused ifdef HAVE_SELINUX_LOGIN_DIR Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#523][comment] IPA: Drop unused ifdef HAVE_SELINUX_LOGIN_DIR

2018-02-27 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/523 Title: #523: IPA: Drop unused ifdef HAVE_SELINUX_LOGIN_DIR pbrezina commented: """ Ack. """ See the full comment at https://github.com/SSSD/sssd/pull/523#issuecomment-368814787 ___ sssd-devel mailing list

[SSSD] [sssd PR#515][comment] sssctl: Showing help even when sssd not configured

2018-02-27 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/515 Title: #515: sssctl: Showing help even when sssd not configured pbrezina commented: """ In short, `poptGetNextOpt` returns next positioned option without value (`-o`) and returns its name (`o`). For example, see: `init_context()` in

[SSSD] [sssd PR#488][comment] Two fixes for certificates in idoverrides

2018-02-27 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/488 Title: #488: Two fixes for certificates in idoverrides sumit-bose commented: """ Yes, only 'ipa: remove SYSDB_USER_CERT from sub-domain users' (4300385) needs to be backported from this PR. The other two commit @jhrozek mentioned are for ssh public

[SSSD] [sssd PR#488][comment] Two fixes for certificates in idoverrides

2018-02-27 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/488 Title: #488: Two fixes for certificates in idoverrides lslebodn commented: """ @sumit-bose, I think it should be enough to backport "ipa: remove SYSDB_USER_CERT from sub-domain users". At least just this patch was in #494. Is that correct? """ See

[SSSD] [sssd PR#488][closed] Two fixes for certificates in idoverrides

2018-02-27 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/488 Author: sumit-bose Title: #488: Two fixes for certificates in idoverrides Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/488/head:pr488 git checkout pr488

[SSSD] [sssd PR#488][comment] Two fixes for certificates in idoverrides

2018-02-27 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/488 Title: #488: Two fixes for certificates in idoverrides lslebodn commented: """ >Yes, but one of the ssh patches was a dependency for 4300385 so I thought it's >safer to just cleanly apply patches and have one less bug in sssd-1-13 at the >same time.

[SSSD] [sssd PR#488][+Pushed] Two fixes for certificates in idoverrides

2018-02-27 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/488 Title: #488: Two fixes for certificates in idoverrides Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#523][closed] IPA: Drop unused ifdef HAVE_SELINUX_LOGIN_DIR

2018-02-27 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/523 Author: lslebodn Title: #523: IPA: Drop unused ifdef HAVE_SELINUX_LOGIN_DIR Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/523/head:pr523 git checkout pr523

[SSSD] [sssd PR#523][+Pushed] IPA: Drop unused ifdef HAVE_SELINUX_LOGIN_DIR

2018-02-27 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/523 Title: #523: IPA: Drop unused ifdef HAVE_SELINUX_LOGIN_DIR Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#523][comment] IPA: Drop unused ifdef HAVE_SELINUX_LOGIN_DIR

2018-02-27 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/523 Title: #523: IPA: Drop unused ifdef HAVE_SELINUX_LOGIN_DIR lslebodn commented: """ master: * 47362caf4c30c3ff5a143f4f4a7fd5599db03b48 * 2f8d0cc83b986b1d302d431f7a3b7c90682faf99 """ See the full comment at