[Freeipa-devel] [PATCH 0042] Removed dead code from LDAPRemoveReverseMember

Jan Cholasta jcholast at redhat.com
Tue Jun 7 07:07:40 UTC 2016


On 6.6.2016 18:29, Martin Basti wrote:
>
>
> On 03.06.2016 14:28, Stanislav Laznicka wrote:
>> On 06/03/2016 02:19 PM, Martin Basti wrote:
>>>
>>> On 03.06.2016 14:13, Stanislav Laznicka wrote:
>>>> https://fedorahosted.org/freeipa/ticket/5892
>>>>
>>>>
>>> NACK
>>>
>>> please remove it from LDAPAddReverseMember too, it contains the same
>>> code
>>>
>>> Martin^2
>>
>> Please see the modified patch.
>>
>> Standa
>>
> ACK
>
> Pushed to master: c56d65b064e1e0410c03cf1206816cad4d8d86cc

I think the attrs_list was supposed to be passed to the ldap.get_entry() 
call rather than removed, which would fix that every reverse member 
command always acts like --all was specified.

-- 
Jan Cholasta




More information about the Freeipa-devel mailing list