<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 03/27/2013 12:15 PM, Tomas Babej
wrote:<br>
</div>
<blockquote cite="mid:5152D4E7.8060703@redhat.com" type="cite">
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
<div class="moz-cite-prefix">On 03/26/2013 07:45 PM, Ana
Krivokapic wrote:<br>
</div>
<blockquote cite="mid:5151ECC0.60005@redhat.com" type="cite">
<pre wrap="">Add the option to create home directories for users on their first login
to ipa-server-install and ipa-replica-install.
<a moz-do-not-send="true" class="moz-txt-link-freetext" href="https://fedorahosted.org/freeipa/ticket/3515">https://fedorahosted.org/freeipa/ticket/3515</a>
</pre>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
Freeipa-devel mailing list
<a moz-do-not-send="true" class="moz-txt-link-abbreviated" href="mailto:Freeipa-devel@redhat.com">Freeipa-devel@redhat.com</a>
<a moz-do-not-send="true" 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>
ACK from the functional point of view.<br>
<br>
Just a nitpick, can you please reformat the patch and fix PEP8
E501 errors?<br>
<br>
I know there's a lot of PEP8 errors and warnings in the whole
project (and we probably need to adress this), however, let us not
introduce new ones.<br>
<br>
Tomas<br>
</blockquote>
<br>
Fixed, thanks.<br>
<pre class="moz-signature" cols="72">--
Regards,
Ana Krivokapic
Associate Software Engineer
FreeIPA team
Red Hat Inc.</pre>
</body>
</html>