[Freeipa-devel] [PATCH] 288 man page for ipa-join

Martin Nagy mnagy at redhat.com
Mon Oct 12 18:46:17 UTC 2009


Rob Crittenden wrote:
> Martin Nagy wrote:
> > On Thu, 2009-10-08 at 11:11 -0400, Rob Crittenden wrote:
> >> Add a man page for the new ipa-join command.
> >>
> >> rob
> > 
> > +ipa\-join [ \fB\-h\fR hostname ] [ \fB\-k\fR keytab\-file ] [ \fB\-w\fR
> > bulk bind password ] [ \fB\-d\fR ] [ \fB\-q\fR ]
> > 
> > Can you use something like bulk-bind-password instead? (with dashes)

Ah, I didn't mean that you should change -w. But rather, I would change
[ \fB\-w\fR bulk bind password ]
to something like this:
[ \fB\-w\fR bulk\-bind\-password ]

Similarly as we use "keytab\-file" and not "keytab file".

> I picked -w since that is common to the openldap clients. I can add a 
> long version too.
> 
> > There is also some trailing white-space in the copyright notice and
> > after all .TP commands.
> 
> I think that the manpage editor I use, manedit, adds those. I can get 
> those cleaned up as well.
> 
> Do you want another patch or just a promise that I'll fix these up?
> 
> rob

Yes, if you do the changes you have an Ack.
Martin




More information about the Freeipa-devel mailing list