fixup for restore with pak (#1883)

rstudio · Oct 10, 2024 · 08a1518 · 08a1518
1 parent 5725cde
commit 08a1518
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/NEWS.md b/NEWS.md
@@ -1,6 +1,10 @@
 
 # renv (development version)
 
+* Fixed an issue where `renv::restore()` could fail when using `pak` if one or
+  more of the packages to be installed declared remotes that were incompatible
+  with the package records recorded in the `renv` lockfile. (#1883)
+
 * Fixed an issue where headers were not properly passed to `available.packages()`
   during `renv` bootstrap. (#1942)
 

diff --git a/R/pak.R b/R/pak.R
@@ -163,10 +163,13 @@ renv_pak_restore <- function(lockfile,
     return(invisible(TRUE))
   }
 
-  # perform installation
+  # perform installation -- use 'dependencies = FALSE' to disable the
+  # pak dependency resolver, as the lockfile should already be complete
+  # https://github.com/rstudio/renv/issues/1883#issuecomment-2404113473
   pak$pkg_install(
     pkg = remotes,
     ask = prompt,
+    dependencies = FALSE
   )
 
   # return installed records