Skip to content

Commit

Permalink
refactor(pkg): simplify depopts passing
Browse files Browse the repository at this point in the history
do not pass entire package when we only need the depopts

Signed-off-by: Rudi Grinberg <[email protected]>

<!-- ps-id: 27050776-a18a-4b63-9bee-195280c83c34 -->
  • Loading branch information
rgrinberg committed Aug 10, 2024
1 parent 9af2225 commit 3f9a48b
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/dune_pkg/opam_solver.ml
Original file line number Diff line number Diff line change
Expand Up @@ -541,12 +541,12 @@ let make_action = function

(* Returns the set of depopts of a package which are part of the solution
represented by [version_by_package_name] *)
let available_depopts solver_env version_by_package_name (opam_package : OpamFile.OPAM.t) =
let available_depopts solver_env version_by_package_name depopts =
let formula =
Resolve_opam_formula.apply_filter
(Solver_env.to_env solver_env)
~with_test:false
opam_package.depopts
depopts
in
let atoms = OpamFormula.to_dnf formula |> List.concat in
List.filter_map atoms ~f:(fun (name, version_constraint) ->
Expand Down Expand Up @@ -647,7 +647,7 @@ let opam_package_to_lock_file_pkg
]
in
let depopts =
available_depopts solver_env version_by_package_name opam_file
available_depopts solver_env version_by_package_name opam_file.depopts
|> List.filter ~f:(fun package_name ->
not (List.mem depends package_name ~equal:Package_name.equal))
in
Expand Down

0 comments on commit 3f9a48b

Please sign in to comment.