[Freeipa-devel] [PATCH] Initial Radius Work

[Pete Rowley]

Karl MacMillan wrote:
> On Sat, 2007-11-03 at 13:59 -0400, John Dennis wrote:
>   
>> Attached are 3 mercurial changesets that comprise the initial work to 
>> integrate freeradius with IPA, please review.
>>
>> Question: It would be easier to review the patch if the diffs were 
>> cumulative rather than sequential (the last two changesets were minor 
>> issues discovered during a test install). Is there a way to get mecurial 
>> to roll up all the changes in a revision range into one diff? I couldn't 
>> figure out how to do it.
>>
>>     
>
> You can either just use hg diff specifying multiple revisions or - what
> I recommend - use mercurial patch queues. 
>
> http://www.selenic.com/mercurial/wiki/index.cgi/MqExtension
>
>   
>> Note: This comprises only the IPA source changes. There is also a 
>> significant change to the freeradius package in order to work with IPA. 
>> Sometime soon I'll build and post a new freeradius rpm. I wanted to get 
>> these initial changes out first for review ASAP.
>>     
>
> I went ahead and pulled in these changes to help with the merging of
> some other patches. A few comments:
>
> 1) Simo / Pete - can you please review the schema added.
>   
Looks good.



-- 
Pete

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