From 927d831612845717c7182cd7f8efc842d1cd0dc9 Mon Sep 17 00:00:00 2001 From: Massimiliano Culpo Date: Wed, 5 Jul 2023 14:11:33 +0200 Subject: Removed leftover TODOs --- lib/spack/spack/solver/asp.py | 17 +++++------------ lib/spack/spack/solver/concretize.lp | 2 -- 2 files changed, 5 insertions(+), 14 deletions(-) diff --git a/lib/spack/spack/solver/asp.py b/lib/spack/spack/solver/asp.py index e8717c9e38..b845750a8c 100644 --- a/lib/spack/spack/solver/asp.py +++ b/lib/spack/spack/solver/asp.py @@ -967,7 +967,6 @@ class SpackSolverSetup: ) for weight, declared_version in enumerate(most_to_least_preferred): - # TODO: self.package_fact(pkg.name).version_declared(declared_version, weight=weight) self.gen.fact( fn.facts( pkg.name, @@ -1360,9 +1359,6 @@ class SpackSolverSetup: cache[imposed_spec] = (effect_id, requirements) effect_id, requirements = cache[imposed_spec] self.gen.fact(fn.facts(named_cond.name, fn.condition_effect(condition_id, effect_id))) - - # FIXME: self.gen.fact(fn.imposed_constraint(condition_id, *predicate.args)) - return condition_id def impose(self, condition_id, imposed_spec, node=True, name=None, body=False): @@ -2658,19 +2654,16 @@ class SpecBuilder: package.update_external_dependencies(self._specs.get(extendee_node, None)) def depends_on(self, parent_node, dependency_node, type): - dependencies = self._specs[parent_node].edges_to_dependencies(name=dependency_node) - - # TODO: assertion to be removed when cross-compilation is handled correctly - msg = "Current solver does not handle multiple dependency edges of the same name" - assert len(dependencies) < 2, msg + dependency_spec = self._specs[dependency_node] + edges = self._specs[parent_node].edges_to_dependencies(name=dependency_spec.name) + edges = [x for x in edges if id(x.spec) == id(dependency_spec)] - if not dependencies: + if not edges: self._specs[parent_node].add_dependency_edge( self._specs[dependency_node], deptypes=(type,), virtuals=() ) else: - # TODO: This assumes that each solve unifies dependencies - dependencies[0].update_deptypes(deptypes=(type,)) + edges[0].update_deptypes(deptypes=(type,)) def virtual_on_edge(self, parent_node, provider_node, virtual): provider = self.extract_pkg(provider_node) diff --git a/lib/spack/spack/solver/concretize.lp b/lib/spack/spack/solver/concretize.lp index 307a8719a8..d2eb849025 100644 --- a/lib/spack/spack/solver/concretize.lp +++ b/lib/spack/spack/solver/concretize.lp @@ -644,7 +644,6 @@ requirement_group_satisfied(node(ID, Package), X) :- % flags if their only source is from a requirement. This is overly-specific % and should use a more-generic approach like in https://github.com/spack/spack/pull/37180 -% FIXME (node transformation): check the following two rules for node(ID, A1) { attr("node_flag", node(ID, A1), A2, A3) } :- requirement_group_member(Y, Package, X), activate_requirement(node(ID, Package), X), @@ -822,7 +821,6 @@ external_with_variant_set(node(NodeID, Package), Variant, Value) external(node(NodeID, Package)), attr("node", node(NodeID, Package)). -% FIXME (node transformation): check how to define the default value % The default value for a variant in a package is what is prescribed: % % 1. On the command line -- cgit v1.2.3-70-g09d2