Do not ask to install pin-depends twice

ocaml · Jan 22, 2025 · afa0d36 · afa0d36
1 parent f80c027
commit afa0d36
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/master_changes.md b/master_changes.md
@@ -47,6 +47,7 @@ users)
 ## Pin
   * [NEW] Make it so pin list display the current revision of a pinned repository in a new column [#6274 @desumn - fix #5533]
   * [BUG] Stop double pin of packages located in ./opam/opam [#6343 @kit-ty-kate - fix #6342]
+  * [BUG] Do not ask to install pin-depends twice [#6375 @kit-ty-kate - fix #6374]
 
 ## List
 

diff --git a/src/client/opamAuxCommands.ml b/src/client/opamAuxCommands.ml
@@ -494,12 +494,6 @@ let autopin st ?(simulate=false) ?quiet ?locked ?recurse ?subpath
     else
       OpamUpdate.dev_packages st ~working_dir:OpamPackage.Set.empty already_pinned
   in
-  let st =
-    if OpamClientConfig.(!r.ignore_pin_depends) then st else
-      OpamPackage.Set.fold (fun nv st ->
-          OpamPinCommand.handle_pin_depends st nv (OpamSwitchState.opam st nv))
-        (OpamPackage.Set.union pins already_pinned_set) st
-  in
   st, atoms
 
 let check_and_revert_sandboxing root config =
-Original file line number
+Diff line change
@@ Expand Up / @@ -47,6 +47,7 @@ users) @@
     ## Pin
       * [NEW] Make it so pin list display the current revision of a pinned repository in a new column [#6274 @desumn - fix #5533]
       * [BUG] Stop double pin of packages located in ./opam/opam [#6343 @kit-ty-kate - fix #6342]
+      * [BUG] Do not ask to install pin-depends twice [#6375 @kit-ty-kate - fix #6374]
     ## List
@@ Expand Down @@