<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 08/20/2015 01:59 PM, Martin Babinsky
wrote:<br>
</div>
<blockquote cite="mid:55D5C125.4030002@redhat.com" type="cite">On
08/20/2015 12:11 PM, Martin Babinsky wrote:
<br>
<blockquote type="cite">On 08/20/2015 11:41 AM, thierry bordaz
wrote:
<br>
<blockquote type="cite">On 08/19/2015 06:28 PM, Martin Babinsky
wrote:
<br>
<blockquote type="cite">On 08/19/2015 02:54 PM, Martin
Babinsky wrote:
<br>
<blockquote type="cite">this patch prevents
<a class="moz-txt-link-freetext" href="https://fedorahosted.org/freeipa/ticket/5234">https://fedorahosted.org/freeipa/ticket/5234</a> from
<br>
happening.
<br>
<br>
<br>
<br>
</blockquote>
Actually, we (myself, mbasti, jcholast) found out that
`user-del
<br>
--preserve` could use some more usability improvements.
<br>
<br>
This quick patch should fix both
<br>
<a class="moz-txt-link-freetext" href="https://fedorahosted.org/freeipa/ticket/5234">https://fedorahosted.org/freeipa/ticket/5234</a> and
<br>
<a class="moz-txt-link-freetext" href="https://fedorahosted.org/freeipa/ticket/5236">https://fedorahosted.org/freeipa/ticket/5236</a> and make user
<br>
preservation operate on multiple arguments in a same way as
plain
<br>
deletion.
<br>
<br>
<br>
<br>
</blockquote>
<br>
Hi Martin,
<br>
<br>
The tests are ok and the fix looks good to me.
<br>
<br>
ACK
<br>
<br>
thanks
<br>
thierry
<br>
</blockquote>
<br>
That's nice, but I have found some small nitpicks and will send
an
<br>
updated version.
<br>
<br>
So self-NACK.
<br>
<br>
</blockquote>
Attaching updated patch.
<br>
<br>
</blockquote>
<font face="Times New Roman, Times, serif"><br>
This is working fine.<br>
<br>
ACK<br>
</font>
</body>
</html>