[Freeipa-devel] [PATCH] 288 man page for ipa-join
Rob Crittenden
rcritten at redhat.com
Mon Oct 12 18:50:19 UTC 2009
Martin Nagy wrote:
> 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
Done and pushed to master
rob
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3245 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20091012/f6d3049a/attachment.bin>
More information about the Freeipa-devel
mailing list