<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 09/20/2013 09:38 AM, Nathaniel
McCallum wrote:<br>
</div>
<blockquote cite="mid:1379695123.1629.10.camel@localhost"
type="cite">
<pre wrap="">On Thu, 2013-09-12 at 16:48 -0400, Nathaniel McCallum wrote:
</pre>
<blockquote type="cite">
<pre wrap="">On Thu, 2013-09-05 at 00:06 -0400, Nathaniel McCallum wrote:
</pre>
<blockquote type="cite">
<pre wrap="">patch attached
</pre>
</blockquote>
<pre wrap="">
Update for ./makeapi attached.
</pre>
</blockquote>
<pre wrap="">
Version 3. This should fix all the current review issues, including the
use of the referential integrity plugin. I had to make one schema change
in order to make the referential integrity modification work. Note also
that the command name prefix is changed from radius to radiusproxy.</pre>
</blockquote>
The LDIF and update related changes look good to me. I'm not
familiar with the ipalib code myself, so someone else should review
that portion of your patch.<br>
<br>
Thanks,<br>
-NGK<br>
<blockquote cite="mid:1379695123.1629.10.camel@localhost"
type="cite">
<pre wrap="">
Nathaniel
</pre>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
Freeipa-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Freeipa-devel@redhat.com">Freeipa-devel@redhat.com</a>
<a class="moz-txt-link-freetext" href="https://www.redhat.com/mailman/listinfo/freeipa-devel">https://www.redhat.com/mailman/listinfo/freeipa-devel</a></pre>
</blockquote>
<br>
</body>
</html>