[Freeipa-devel] [PATCH 0060] raise an error when trying to preserve an already preserved user

[Martin Basti]

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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20150825/d40d8fd3/attachment.htm>