URL: https://github.com/SSSD/sssd/pull/988 Author: miztake Title: #988: man: fix typos Action: opened
PR body: """ - add whitespace between words - correct manpage reference - correct wrong word - capitalize the first letter """
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/988/head:pr988 git checkout pr988
URL: https://github.com/SSSD/sssd/pull/988 Title: #988: man: fix typos
Label: +Changes requested
URL: https://github.com/SSSD/sssd/pull/988 Author: miztake Title: #988: man: fix typos Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/988/head:pr988 git checkout pr988
URL: https://github.com/SSSD/sssd/pull/988 Title: #988: man: fix typos
pbrezina commented: """ Thank you. May I ask you what distribution do you use? Does this happen when you compile the sources yourself or is the mistake bundled in the distributed sssd package? """
See the full comment at https://github.com/SSSD/sssd/pull/988#issuecomment-590276428
URL: https://github.com/SSSD/sssd/pull/988 Title: #988: man: fix typos
miztake commented: """ Hi, @pbrezina Thank you for your comment.
May I ask you what distribution do you use?
I use CentOS 7. I built sssd on CentOS 7.
# cat /etc/redhat-release CentOS Linux release 7.7.1908 (Core)
Does this happen when you compile the sources yourself or is the mistake bundled in the distributed sssd package?
This happen when I compile the sources myself. Packages provided by CentOS have similar results.
The following is the result on CentOS 7.
# man sss_ssh_authorizedkeys <snip> KEYS FROM CERTIFICATES In addition to the public SSH keys for user USERsss_ssh_authorizedkeys can return public SSH keys derived from the public key of a X.509 certificate as well.
# man sssd-ldap DESCRIPTION <snip> LDAP back end supports id, auth, access and chpass providers. If you want to authenticate against an LDAP server either TLS/SSL or LDAPS is required. sssddoes not support """
See the full comment at https://github.com/SSSD/sssd/pull/988#issuecomment-590930540
URL: https://github.com/SSSD/sssd/pull/988 Title: #988: man: fix typos
Label: -Changes requested
URL: https://github.com/SSSD/sssd/pull/988 Title: #988: man: fix typos
Label: +Waiting for review
URL: https://github.com/SSSD/sssd/pull/988 Title: #988: man: fix typos
pbrezina commented: """ Ok, I'll check how it looks on centos and why. """
See the full comment at https://github.com/SSSD/sssd/pull/988#issuecomment-591366265
URL: https://github.com/SSSD/sssd/pull/988 Title: #988: man: fix typos
pbrezina commented: """ I checked and the space is missing on both CentOS7 and RHEL7, however it works of at least Fedora 31 and RHEL8. The man page is created using `libxslt`:
``` /usr/bin/xsltproc -o sssd-ldap.5 --catalogs --xinclude --nonet --stringparam profile.condition "with_false;with_sudo;with_autofs;with_ssh;with_pac_responder;with_ifp;gpo_default_enforcing;have_systemd;no_enable_files_domain;with_kcm;with_nss" http://docbook.sourceforge.net/release/xsl/current/manpages/profile-docbook.... sssd-ldap.5.xml ```
So there was probably a bug in `xsltproc` that is already fixed.
I'm going to ack your current patch. If you want the spaced to be fixed on CentOS7 you probably need to file bug report there to rebase `libxslt. """
See the full comment at https://github.com/SSSD/sssd/pull/988#issuecomment-593882083
URL: https://github.com/SSSD/sssd/pull/988 Title: #988: man: fix typos
Label: -Waiting for review
URL: https://github.com/SSSD/sssd/pull/988 Title: #988: man: fix typos
Label: +Ready to push
URL: https://github.com/SSSD/sssd/pull/988 Title: #988: man: fix typos
Label: +Accepted
URL: https://github.com/SSSD/sssd/pull/988 Title: #988: man: fix typos
miztake commented: """ Hi, @pbrezina
Thank you for your confirmation. I understood that missing spaces is due to my environment. """
See the full comment at https://github.com/SSSD/sssd/pull/988#issuecomment-594046081
URL: https://github.com/SSSD/sssd/pull/988 Title: #988: man: fix typos
pbrezina commented: """ * `master` * 9ccf78dbdc7d05b29dcfadff113c4e7064e84bc7 - man: fix typos - correct manpage reference - correct wrong word - capitalize the first letter
"""
See the full comment at https://github.com/SSSD/sssd/pull/988#issuecomment-594474947
URL: https://github.com/SSSD/sssd/pull/988 Title: #988: man: fix typos
Label: +Pushed
URL: https://github.com/SSSD/sssd/pull/988 Title: #988: man: fix typos
Label: -Accepted
URL: https://github.com/SSSD/sssd/pull/988 Title: #988: man: fix typos
Label: -Ready to push
URL: https://github.com/SSSD/sssd/pull/988 Author: miztake Title: #988: man: fix typos Action: closed
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/988/head:pr988 git checkout pr988
sssd-devel@lists.fedorahosted.org
-
miztake -
pbrezina