diff options
author | Zac Medico <zmedico@gentoo.org> | 2011-02-12 23:36:07 -0800 |
---|---|---|
committer | Zac Medico <zmedico@gentoo.org> | 2011-02-12 23:36:07 -0800 |
commit | 37b9c61f2d16d09a4ae1bcb61d63af4e64b667f4 (patch) | |
tree | e5b99c6bd340285c5ae13977fa856214e70898fa /pym/_emerge | |
parent | c42004eb552c9117d221b0e2063e8696efca35dc (diff) | |
download | portage-37b9c61f2d16d09a4ae1bcb61d63af4e64b667f4.tar.gz portage-37b9c61f2d16d09a4ae1bcb61d63af4e64b667f4.tar.bz2 portage-37b9c61f2d16d09a4ae1bcb61d63af4e64b667f4.zip |
depgraph: avoid atom hash collisions in dep_check
Atoms are stored in the graph as (atom, id(atom)) tuples since each
atom is considered to be a unique entity. For example, atoms that
appear identical may behave differently in USE matching, depending on
their unevaluated form. Also, specially generated virtual atoms may
appear identical while having different _orig_atom attributes.
Diffstat (limited to 'pym/_emerge')
-rw-r--r-- | pym/_emerge/depgraph.py | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/pym/_emerge/depgraph.py b/pym/_emerge/depgraph.py index e57176349..bec9ffd1b 100644 --- a/pym/_emerge/depgraph.py +++ b/pym/_emerge/depgraph.py @@ -2146,7 +2146,7 @@ class depgraph(object): if parent is None: selected_atoms = mycheck[1] else: - chosen_atoms = frozenset(mycheck[1]) + chosen_atom_ids = frozenset(id(atom) for atom in mycheck[1]) selected_atoms = OrderedDict() node_stack = [(parent, None, None)] traversed_nodes = set() @@ -2169,13 +2169,14 @@ class depgraph(object): depth=node.depth, parent=node_parent, priority=node_priority, root=node.root) - child_atoms = atom_graph.child_nodes(node) - selected_atoms[k] = [atom for atom in \ - child_atoms if atom in chosen_atoms] - for child_atom in child_atoms: - if child_atom not in chosen_atoms: + child_atoms = [] + selected_atoms[k] = child_atoms + for atom_node in atom_graph.child_nodes(node): + child_atom = atom_node[0] + if id(child_atom) not in chosen_atom_ids: continue - for child_node in atom_graph.child_nodes(child_atom): + child_atoms.append(child_atom) + for child_node in atom_graph.child_nodes(atom_node): if child_node in traversed_nodes: continue if not portage.match_from_list( |