Skip to content

Commit

Permalink
Remove special case for running solver with no targets
Browse files Browse the repository at this point in the history
The TODO deleted in 4d3f7b5 suggests that the
special case was only needed for the top-down solver, which has been removed.
  • Loading branch information
grayjay committed Jul 31, 2023
1 parent 9adf58c commit e0d283b
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions cabal-install/src/Distribution/Client/Dependency.hs
Original file line number Diff line number Diff line change
Expand Up @@ -777,12 +777,6 @@ resolveDependencies
-> Solver
-> DepResolverParams
-> Progress String String SolverInstallPlan
-- TODO: is this needed here? see dontInstallNonReinstallablePackages
resolveDependencies platform comp _pkgConfigDB _solver params
| Set.null (depResolverTargets params) =
return (validateSolverResult platform comp indGoals [])
where
indGoals = depResolverIndependentGoals params
resolveDependencies platform comp pkgConfigDB solver params =
Step (showDepResolverParams finalparams) $
fmap (validateSolverResult platform comp indGoals) $
Expand Down

0 comments on commit e0d283b

Please sign in to comment.