In dblink.treewalk(), tweak the "Safely unmerging

already-installed instance" logic to make sure that
it works in all possible cases. (trunk r8801)

svn path=/main/branches/2.1.2/; revision=8802

diff --git a/pym/portage.py b/pym/portage.py
index ca59746754b8e9defe42a207c4f0b91c5dba81ff..f46276968e23afbdb25c4263428c0620d8892d73 100644 (file)
--- a/pym/portage.py
+++ b/pym/portage.py
@@ -8856,6 +8856,16 @@ class dblink:
                outfile.flush()
                outfile.close()
 
+               # These caches are populated during collision-protect and the data
+               # they contain is now invalid. It's very important to invalidate
+               # the contents_inodes cache so that FEATURES=unmerge-orphans
+               # doesn't unmerge anything that belongs to this package that has
+               # just been merged.
+               others_in_slot.append(self)  # self has just been merged
+               for dblnk in others_in_slot:
+                       dblnk.contentscache = None
+                       dblnk._contents_inodes = None
+
                # If portage is reinstalling itself, remove the old
                # version now since we want to use the temporary
                # PORTAGE_BIN_PATH that will be removed when we return.
@@ -8865,19 +8875,13 @@ class dblink:
                        "portage" == pkgsplit(self.pkg)[0]:
                        reinstall_self = True
 
-               for dblnk in others_in_slot:
+               for dblnk in list(others_in_slot):
+                       if dblnk is self:
+                               continue
                        if dblnk.mycpv != self.mycpv and \
                                not reinstall_self:
                                continue
                        writemsg_stdout(">>> Safely unmerging already-installed instance...\n")
-                       # These caches are populated during collision-protect and the data
-                       # they contain is now invalid. It's very important to invalidate
-                       # the contents_inodes cache so that FEATURES=unmerge-orphans
-                       # doesn't unmerge anything that belongs to this package that has
-                       # just been merged.
-                       self.contentscache = None
-                       self._contents_inodes = None
-                       others_in_slot.append(self)  # self has just been merged
                        others_in_slot.remove(dblnk) # dblnk will unmerge itself now
                        dblnk.unmerge(trimworld=0, ldpath_mtimes=prev_mtimes,
                                others_in_slot=others_in_slot)