Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

man ipa-cacert-manage install needs clarification #635

Closed
wants to merge 1 commit into from

Conversation

flo-renaud
Copy link
Contributor

@flo-renaud flo-renaud commented Mar 22, 2017

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

@tkrizek tkrizek self-assigned this Mar 22, 2017
@tkrizek
Copy link
Contributor

tkrizek commented Mar 22, 2017

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?

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
@flo-renaud
Copy link
Contributor Author

Hi @tomaskrizek
thank you for the suggestion. PR updated.

@tkrizek tkrizek added the ack Pull Request approved, can be merged label Mar 22, 2017
@tkrizek
Copy link
Contributor

tkrizek commented Mar 22, 2017

master:

  • 3ea2834 man ipa-cacert-manage install needs clarification
    ipa-4-5:

  • bb53a9a man ipa-cacert-manage install needs clarification

@tkrizek tkrizek added the pushed Pull Request has already been pushed label Mar 22, 2017
@tkrizek tkrizek closed this Mar 22, 2017
@flo-renaud flo-renaud deleted the manpage branch April 11, 2017 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
2 participants