Skip to content
  • kristenk's avatar
    Merge dependencies more effectively in D.Solver.Modular.IndexConversion. · 59febb17
    kristenk authored
    This commit is a more thorough fix for the problem that
    4f7ac10d attempted to fix.  4f7ac10d combined
    all dependencies on the same package within the same build-depends or
    build-tool-depends field, in order to avoid performance problems in the solver
    QuickCheck tests.  That wasn't enough to prevent the solver from creating many
    duplicate dependencies when converting the PackageDescription to the
    solver-specific format, though.  When a package contained many conditionals that
    each contained the same dependency in both branches, the solver lifted those
    dependencies out of the conditionals without deduplicating them.
    
    This commit moves the step that merges dependencies after the step that lifts
    common dependencies out of conditionals.
    59febb17