On 08/21/2015 10:37 AM, thierry bordaz wrote:
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


Pushed to:
master: c6299a8cfde7d4e4bb9a50e3cf6406667cee0a6f
ipa-4-2: 361a4fb4100824b27b777c27c329d50361ba69f4

-- 
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