On 08/20/2015 01:59 PM, Martin Babinsky wrote:
On 08/20/2015 12:11 PM, Martin Babinsky wrote:
On 08/20/2015 11:41 AM, thierry bordaz wrote:
On 08/19/2015 06:28 PM, Martin Babinsky wrote:
On 08/19/2015 02:54 PM, Martin Babinsky wrote:
this patch prevents https://fedorahosted.org/freeipa/ticket/5234 from
happening.



Actually, we (myself, mbasti, jcholast) found out that `user-del
--preserve` could use some more usability improvements.

This quick patch should fix both
https://fedorahosted.org/freeipa/ticket/5234 and
https://fedorahosted.org/freeipa/ticket/5236 and make user
preservation operate on multiple arguments in a same way as plain
deletion.




Hi Martin,

The tests are ok and the fix looks good to me.

ACK

thanks
thierry

That's nice, but I have found some small nitpicks and will send an
updated version.

So self-NACK.

Attaching updated patch.


This is working fine.

ACK
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to