[Freeipa-devel] [PATCH] Add more manpages

Stephen Gallagher sgallagh at redhat.com
Fri May 29 12:43:17 UTC 2009


On 05/29/2009 08:34 AM, Jakub Hrozek wrote:
> On Fri, 2009-05-29 at 08:04 -0400, Stephen Gallagher wrote:
>> The manpage for sssd references sssd.conf(5), but that manpage isn't
>> yet
>> available.
>>
> 
> I know, I just wanted to speed up the process by sending what I had
> complete at the time.

No problem, just wanted to make sure you hadn't forgotten to include it.

> 
>> I've attached a patch that applies atop yours with some grammatical
>> corrections and a few other minor changes. I reasoned it made sense to
>> create a separate patch to more easily tell what changes I was
>> proposing. For the record, your English is better than most native
>> speakers that I know.
> 
> Thank you for the review, I agree with all the changes. Should I resend
> with your changes squashed in or is it OK to keep both?
> 
> Jakub

Squash them together. There's no real benefit to having them separate.



-- 
Stephen Gallagher
RHCE 804006346421761

Looking to carve out IT costs?
www.redhat.com/carveoutcosts/

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 3258 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20090529/da2d704f/attachment.p7s>


More information about the Freeipa-devel mailing list