[Freeipa-devel] [PATCH 0007] Limit memberOf and refInt DS plugins to main IPA suffix

[Petr Viktorin]

On 01/27/2014 01:06 PM, Jan Cholasta wrote:
> Hi,
>
> On 23.1.2014 12:32, Petr Spacek wrote:
>> Hello,
>>
>> this patch solves performance problems with retro change log trimming.
>>
>> It has to go to the same branches as Ana's patch 0079 Enable Retro
>> Changelog and Content Synchronization DS plugins.
>
> Nitpicks:
>
> -Requires: 389-ds-base >= 1.3.1.3
> +Requires: 389-ds-base >= 1.3.2.10
>
> Why 1.3.2.10? According to 389-ds-base changelog, all the required
> enhancements should be available in 1.3.2.9:
>
> https://fedorahosted.org/389/ticket/47526
> https://fedorahosted.org/389/ticket/47527
> https://fedorahosted.org/389/ticket/47621

According to Petr Špaček things were broken in .9 so we do need the .10

>
>
> -# Enable Retro changelog
> +# Enable Retro changelog - is is necessary for SyncRepl
>
> s/is is/it is/
>
>
> Otherwise ACK.

Fixed the typo and pushed to master: 
04627b72d6d6cbf3a9dadc614a532505e31957f5



-- 
Petr³