From 2e77c80a5be2664860aa022f9d681b71e3f8d540 Mon Sep 17 00:00:00 2001 From: Chris Black Date: Fri, 1 Dec 2023 16:06:13 -0800 Subject: [PATCH] Makefile: install with remotes::install_local instead of devtools::install devtools::install insists in checking for updates to all dependencies every call, even when `upgrade = FALSE`. Across all packages this adds up to a lot of repeated checks, and in combination with our separate dependency check it's enough to exceed GitHub APi request rate limits during CI builds. remotes::install_local appears to not check dependencies at all when dependencies=FALSE, which is what we want for this case. --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index a441991063c..cda4ad6f72b 100644 --- a/Makefile +++ b/Makefile @@ -74,7 +74,7 @@ depends_R_pkg = ./scripts/time.sh "depends ${1}" ./scripts/confirm_deps.R ${1} \ $(if $(findstring modules/benchmark,$(1)),NA,TRUE) install_R_pkg = ./scripts/time.sh "install ${1}" Rscript \ -e ${SETROPTIONS} \ - -e "devtools::install('$(strip $(1))', upgrade=FALSE)" + -e "remotes::install_local('$(strip $(1))', force=TRUE, dependencies=FALSE, upgrade=FALSE)" check_R_pkg = ./scripts/time.sh "check ${1}" Rscript scripts/check_with_errors.R $(strip $(1)) test_R_pkg = ./scripts/time.sh "test ${1}" Rscript \ -e "devtools::test('$(strip $(1))'," \