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

Minor typo fixes #707

Closed
wants to merge 1 commit into from
Closed

Minor typo fixes #707

wants to merge 1 commit into from

Conversation

Akasurde
Copy link
Member

@Akasurde Akasurde commented Apr 11, 2017

Fixes https://pagure.io/freeipa/issue/6865

Signed-off-by: Abhijeet Kasurde akasurde@redhat.com

@MartinBasti MartinBasti added the ack Pull Request approved, can be merged label Apr 11, 2017
@stlaz stlaz removed the ack Pull Request approved, can be merged label Apr 11, 2017
@@ -76,7 +76,7 @@ change the name. Please note that changing the NetBIOS name might break
existing trust relationships to other domains.
.TP
\fB\-\-add\-sids\fR
Add SIDs to existing users and groups as on of final steps of the
Add SIDs to existing users and groups as one of the final steps of
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The "the" should have stayed there, it's related to the "run" rather than ipa-adtrust-install.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@@ -76,7 +76,7 @@ change the name. Please note that changing the NetBIOS name might break
existing trust relationships to other domains.
.TP
\fB\-\-add\-sids\fR
Add SIDs to existing users and groups as on of final steps of the
Add SIDs to existing users and groups as one of final steps of the
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-Add SIDs to existing users and groups as on of final steps of the
+Add SIDs to existing users and groups as one of the final steps of the

Fixes https://pagure.io/freeipa/issue/6865

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
@stlaz
Copy link
Contributor

stlaz commented Apr 11, 2017

Thanks, ACK.

@stlaz stlaz added the ack Pull Request approved, can be merged label Apr 11, 2017
@tkrizek tkrizek added the pushed Pull Request has already been pushed label Apr 12, 2017
@tkrizek
Copy link
Contributor

tkrizek commented Apr 12, 2017

master:

@tkrizek tkrizek closed this Apr 12, 2017
@Akasurde
Copy link
Member Author

@stlaz @tomaskrizek Thanks.

@Akasurde Akasurde deleted the manad branch April 12, 2017 17:06
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
4 participants