Install before building pkgdown, closes #89

ropensci · Sep 13, 2018 · 9e2ccdf · 9e2ccdf
1 parent e383b2e
commit 9e2ccdf
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 11 deletions.
diff --git a/R/steps-pkgdown.R b/R/steps-pkgdown.R
@@ -9,15 +9,12 @@ BuildPkgdown <- R6Class(
     },
 
     run = function() {
-      # Don't need to be super-strict when building pkgdown
-      withr::with_libpaths(
-        super$get_lib(), action = "prefix",
-        do.call(pkgdown::build_site, c(list(preview = FALSE), private$pkgdown_args))
-      )
+      remotes::install_local(".")
+      do.call(pkgdown::build_site, c(list(preview = FALSE), private$pkgdown_args))
     },
 
     prepare = function() {
-      verify_install("pkgdown")
+      verify_install(c("pkgdown", "remotes"))
       super$prepare()
     }
   ),

diff --git a/R/steps-rcmdcheck.R b/R/steps-rcmdcheck.R
@@ -8,15 +8,12 @@ TicStepWithPackageDeps <- R6Class(
       verify_install("remotes")
 
       remotes::install_deps(dependencies = TRUE)
-      utils::update.packages(lib.loc = self$get_lib()[[1]], ask = FALSE)
-    },
 
-    get_lib = function() {
       # Using a separate library for "build dependencies"
       # (which might well be ahead of CRAN)
       # works very poorly with custom steps that are not aware
       # of this shadow library.
-      .libPaths()
+      utils::update.packages(ask = FALSE)
     }
   ),
 )
@@ -35,7 +32,7 @@ RCMDcheck <- R6Class(
     },
 
     run = function() {
-      res <- rcmdcheck::rcmdcheck(args = private$args, libpath = super$get_lib())
+      res <- rcmdcheck::rcmdcheck(args = private$args)
 
       print(res)
       if (length(res$errors) > 0) {