On Tue, 2012-03-06 at 13:55 +0100, Petr Viktorin wrote:
> On 03/05/2012 04:10 PM, Petr Viktorin wrote:
> > This patch fixes DN handling when removing LDAP entries from groups.
> >
> > Because they deal with commas and backslashes in a CSV param, the tests
> > here rely on my patch 0015.
> >
> >
> >
> > _______________________________________________
> > Freeipa-devel mailing list
> > Freeipa-devel@redhat.com
> > https://www.redhat.com/mailman/listinfo/freeipa-devel
> 
> Here is the same patch with CSV-stuff-dependent tests removed.
> 

Thanks, this patch looks and works fine, so ACK.

Before we push it, can you please update the tests in this patch or even
better send an updated version of your patch 22 where RDN is changed to
ipaUniqueID? Otherwise, if we push both these patches, we would get test
failures with DN mismatches.

Martin

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to