Skip to content

Commit 0f1f67c

Browse files
mpickeringMikolaj
authored andcommitted
Refactor resolveTargets to allow resolving targets from a PackageDescription
This commit refactors `resolveTargets` to allow you to either resolve targets from * The output of the solver by `resolveTargetsFromSolver` * The package descriptions `resolveTargetsFromLocalPackages` This allows you to use `v2-` infrastructure with commands which don't require running the solver step (for example, v2-outdated).
1 parent c947766 commit 0f1f67c

File tree

14 files changed

+182
-40
lines changed

14 files changed

+182
-40
lines changed

cabal-install/src/Distribution/Client/CmdBench.hs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ benchAction flags@NixStyleFlags{..} targetStrings globalFlags = do
132132
-- (as opposed to say build or haddock targets).
133133
targets <-
134134
either (reportTargetProblems verbosity) return $
135-
resolveTargets
135+
resolveTargetsFromSolver
136136
selectPackageTargets
137137
selectComponentTarget
138138
elaboratedPlan

cabal-install/src/Distribution/Client/CmdBuild.hs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,7 @@ buildAction flags@NixStyleFlags{extraFlags = buildFlags, ..} targetStrings globa
157157
-- (as opposed to say repl or haddock targets).
158158
targets <-
159159
either (reportBuildTargetProblems verbosity) return $
160-
resolveTargets
160+
resolveTargetsFromSolver
161161
selectPackageTargets
162162
selectComponentTarget
163163
elaboratedPlan

cabal-install/src/Distribution/Client/CmdGenBounds.hs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ genBoundsAction flags targetStrings globalFlags =
109109
-- Step 2: Resolve the targets for the gen-bounds command.
110110
targets <-
111111
either (reportGenBoundsTargetProblems verbosity) return $
112-
resolveTargets
112+
resolveTargetsFromSolver
113113
selectPackageTargets
114114
selectComponentTarget
115115
elaboratedPlan

cabal-install/src/Distribution/Client/CmdHaddock.hs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -177,7 +177,7 @@ haddockAction relFlags targetStrings globalFlags = do
177177
-- haddock targets
178178
targets <-
179179
either (reportBuildDocumentationTargetProblems verbosity) return $
180-
resolveTargets
180+
resolveTargetsFromSolver
181181
(selectPackageTargets haddockFlags)
182182
selectComponentTarget
183183
elaboratedPlan

cabal-install/src/Distribution/Client/CmdHaddockProject.hs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ import Distribution.Client.ProjectOrchestration
2828
, ProjectBuildContext (..)
2929
, TargetSelector (..)
3030
, pruneInstallPlanToTargets
31-
, resolveTargets
31+
, resolveTargetsFromSolver
3232
, runProjectPreBuildPhase
3333
, selectComponentTargetBasic
3434
)
@@ -146,7 +146,7 @@ haddockProjectAction flags _extraArgs globalFlags = do
146146
-- (as opposed to say repl or haddock targets).
147147
targets <-
148148
either reportTargetProblems return $
149-
resolveTargets
149+
resolveTargetsFromSolver
150150
selectPackageTargets
151151
selectComponentTargetBasic
152152
elaboratedPlan

cabal-install/src/Distribution/Client/CmdInstall.hs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -828,7 +828,7 @@ partitionToKnownTargetsAndHackagePackages
828828
-> IO (TargetsMap, [PackageName])
829829
partitionToKnownTargetsAndHackagePackages verbosity pkgDb elaboratedPlan targetSelectors = do
830830
let mTargets =
831-
resolveTargets
831+
resolveTargetsFromSolver
832832
selectPackageTargets
833833
selectComponentTarget
834834
elaboratedPlan
@@ -868,7 +868,7 @@ partitionToKnownTargetsAndHackagePackages verbosity pkgDb elaboratedPlan targetS
868868
-- removed (or we've given up).
869869
targets <-
870870
either (reportBuildTargetProblems verbosity) return $
871-
resolveTargets
871+
resolveTargetsFromSolver
872872
selectPackageTargets
873873
selectComponentTarget
874874
elaboratedPlan
@@ -888,7 +888,7 @@ constructProjectBuildContext verbosity baseCtx targetSelectors = do
888888
-- Interpret the targets on the command line as build targets
889889
targets <-
890890
either (reportBuildTargetProblems verbosity) return $
891-
resolveTargets
891+
resolveTargetsFromSolver
892892
selectPackageTargets
893893
selectComponentTarget
894894
elaboratedPlan

cabal-install/src/Distribution/Client/CmdListBin.hs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ listbinAction flags@NixStyleFlags{..} args globalFlags = do
106106
-- (as opposed to say repl or haddock targets).
107107
targets <-
108108
either (reportTargetProblems verbosity) return $
109-
resolveTargets
109+
resolveTargetsFromSolver
110110
selectPackageTargets
111111
selectComponentTarget
112112
elaboratedPlan

cabal-install/src/Distribution/Client/CmdRepl.hs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -516,7 +516,7 @@ replAction flags@NixStyleFlags{extraFlags = r@ReplFlags{..}, ..} targetStrings g
516516
-- (as opposed to say build or haddock targets).
517517
targets <-
518518
either (reportTargetProblems verbosity) return $
519-
resolveTargets
519+
resolveTargetsFromSolver
520520
(selectPackageTargets multi_repl_enabled)
521521
selectComponentTarget
522522
elaboratedPlan

cabal-install/src/Distribution/Client/CmdRun.hs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,7 @@ runAction flags@NixStyleFlags{..} targetAndArgs globalFlags =
228228
-- (as opposed to say repl or haddock targets).
229229
targets <-
230230
either (reportTargetProblems verbosity) return $
231-
resolveTargets
231+
resolveTargetsFromSolver
232232
selectPackageTargets
233233
selectComponentTarget
234234
elaboratedPlan

cabal-install/src/Distribution/Client/CmdTarget.hs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ targetAction flags@NixStyleFlags{..} ts globalFlags = do
172172

173173
targets :: TargetsMap <-
174174
either (reportBuildTargetProblems verbosity) return $
175-
resolveTargets
175+
resolveTargetsFromSolver
176176
selectPackageTargets
177177
selectComponentTarget
178178
elaboratedPlan

0 commit comments

Comments
 (0)