Discussion:
[Freeipa-devel] [freeipa PR#635][opened] man ipa-cacert-manage install needs clarification
flo-renaud
2017-03-22 07:59:39 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/635
Author: flo-renaud
Title: #635: man ipa-cacert-manage install needs clarification
Action: opened

PR body:
"""
The customers are often confused by ipa-cacert-manage install. The man page
should make it clear that IPA CA is not modified in any way by this command.

https://pagure.io/freeipa/issue/6795
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/635/head:pr635
git checkout pr635
tomaskrizek
2017-03-22 08:42:36 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/635
Title: #635: man ipa-cacert-manage install needs clarification

tomaskrizek commented:
"""
I think the message would be a bit easier to read if the added part would be in a separate paragraph. Could you update it, please?
"""

See the full comment at https://github.com/freeipa/freeipa/pull/635#issuecomment-288332977
flo-renaud
2017-03-22 08:57:04 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/635
Author: flo-renaud
Title: #635: man ipa-cacert-manage install needs clarification
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/635/head:pr635
git checkout pr635
flo-renaud
2017-03-22 08:58:11 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/635
Title: #635: man ipa-cacert-manage install needs clarification

flo-renaud commented:
"""
Hi @tomaskrizek
thank you for the suggestion. PR updated.
"""

See the full comment at https://github.com/freeipa/freeipa/pull/635#issuecomment-288336324
tomaskrizek
2017-03-22 09:00:48 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/635
Title: #635: man ipa-cacert-manage install needs clarification

Label: +ack
tomaskrizek
2017-03-22 09:15:05 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/635
Title: #635: man ipa-cacert-manage install needs clarification

Label: +pushed
tomaskrizek
2017-03-22 09:15:12 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/635
Title: #635: man ipa-cacert-manage install needs clarification

tomaskrizek commented:
"""
master:

* 3ea2834b76a72c97186b01487e885800754c0fbc man ipa-cacert-manage install needs clarification
ipa-4-5:

* bb53a9ab6dce023dd51c2a434fd8597eab5bc0d0 man ipa-cacert-manage install needs clarification
"""

See the full comment at https://github.com/freeipa/freeipa/pull/635#issuecomment-288340038
tomaskrizek
2017-03-22 09:15:15 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/635
Author: flo-renaud
Title: #635: man ipa-cacert-manage install needs clarification
Action: closed

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/635/head:pr635
git checkout pr635

Loading...