On 27/04/15 18:42, David Kupka wrote:
On 04/16/2015 05:14 PM, Martin Basti wrote:
https://fedorahosted.org/freeipa/ticket/4904

Patch attached




I guess the rest of the comment is also outdated. Can you update it, too?

Updated patch attached.

--
Martin Basti

From d43c60e11eb61700e702bb968362d5fa23b5e782 Mon Sep 17 00:00:00 2001
From: Martin Basti <mba...@redhat.com>
Date: Thu, 16 Apr 2015 15:32:01 +0200
Subject: [PATCH] Server Upgrade: fix a comment in ldapupdater

DN sorting was removed in previous patches

https://fedorahosted.org/freeipa/ticket/4904
---
 ipaserver/install/ldapupdate.py | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/ipaserver/install/ldapupdate.py b/ipaserver/install/ldapupdate.py
index 1f30f4b215383a7964c578975b239ea4a57f68a8..171cb904f58ea415e12932dec8f38c02d9e1e688 100644
--- a/ipaserver/install/ldapupdate.py
+++ b/ipaserver/install/ldapupdate.py
@@ -769,12 +769,7 @@ class LDAPUpdate:
 
     def _delete_record(self, updates):
         """
-        Run through all the updates again looking for any that should be
-        deleted.
-
-        This must use a reversed list so that the longest entries are
-        considered first so we don't end up trying to delete a parent
-        and child in the wrong order.
+        Delete record
         """
 
         dn = updates['dn']
-- 
2.1.0

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