[sssd PR#831][opened] MAN: ldap_user_home_directory default missing

[thalman]

   URL: https://github.com/SSSD/sssd/pull/831
Author: thalman
 Title: #831: MAN: ldap_user_home_directory default missing
Action: opened

PR body:
"""
The default value of "ldap_user_home_directory" is "homeDirectory"
but for AD provider it is "unixHomeDirectory"

Resolves:
https://bugzilla.redhat.com/show_bug.cgi?id=1673443
"""

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/831/head:pr831
git checkout pr831

[sssd PR#831][comment] MAN: ldap_user_home_directory default missing

[mzidek-rh]

  URL: https://github.com/SSSD/sssd/pull/831
Title: #831: MAN: ldap_user_home_directory default missing

mzidek-rh commented:
"""
Just a nitpick. Please also indicate that the homeDirectory is default for IPA too, so
that readers will not get confused. For example like this:
```
Default: homeDirectory (LDAP and IPA), unixHomeDirectory (AD)
```
"""

See the full comment at https://github.com/SSSD/sssd/pull/831#issuecomment-503465322

[sssd PR#831][synchronized] MAN: ldap_user_home_directory default missing

[thalman]

   URL: https://github.com/SSSD/sssd/pull/831
Author: thalman
 Title: #831: MAN: ldap_user_home_directory default missing
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/831/head:pr831
git checkout pr831

[sssd PR#831][comment] MAN: ldap_user_home_directory default missing

[thalman]

  URL: https://github.com/SSSD/sssd/pull/831
Title: #831: MAN: ldap_user_home_directory default missing

thalman commented:
"""
>  Just a nitpick. Please also indicate that the homeDirectory is
> default for IPA too, so that readers will not get confused
>  
done
"""

See the full comment at https://github.com/SSSD/sssd/pull/831#issuecomment-503467152

[sssd PR#831][comment] MAN: ldap_user_home_directory default missing

[mzidek-rh]

  URL: https://github.com/SSSD/sssd/pull/831
Title: #831: MAN: ldap_user_home_directory default missing

mzidek-rh commented:
"""
Thanks. Ack.
"""

See the full comment at https://github.com/SSSD/sssd/pull/831#issuecomment-503467689

[sssd PR#831][+Accepted] MAN: ldap_user_home_directory default missing

[mzidek-rh]

  URL: https://github.com/SSSD/sssd/pull/831
Title: #831: MAN: ldap_user_home_directory default missing

Label: +Accepted

[sssd PR#831][comment] MAN: ldap_user_home_directory default missing

[jhrozek]

  URL: https://github.com/SSSD/sssd/pull/831
Title: #831: MAN: ldap_user_home_directory default missing

jhrozek commented:
"""
* master: 01ea70fa8cc91f05a726d1dea3c64bd776dc3517
"""

See the full comment at https://github.com/SSSD/sssd/pull/831#issuecomment-504135449

[sssd PR#831][closed] MAN: ldap_user_home_directory default missing

[jhrozek]

   URL: https://github.com/SSSD/sssd/pull/831
Author: thalman
 Title: #831: MAN: ldap_user_home_directory default missing
Action: closed

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/831/head:pr831
git checkout pr831

[sssd PR#831][+Pushed] MAN: ldap_user_home_directory default missing

[jhrozek]

  URL: https://github.com/SSSD/sssd/pull/831
Title: #831: MAN: ldap_user_home_directory default missing

Label: +Pushed

[sssd PR#831][comment] MAN: ldap_user_home_directory default missing

[jhrozek]

  URL: https://github.com/SSSD/sssd/pull/831
Title: #831: MAN: ldap_user_home_directory default missing

jhrozek commented:
"""
btw only when I pushed the patch did I notice it referenced a RHBZ and not an upstream
ticket. In most cases, a ticket is preferable, please file one and link it with the BZ
next time. (We have an internal tool do to that)
"""

See the full comment at https://github.com/SSSD/sssd/pull/831#issuecomment-504135778