BUG FIX: _R_CHECK_LIMIT_CORES_ was incorrectly set to TRUE when package was loaded in Positron Console [#139]

Author: HenrikBengtsson

DESCRIPTION

@@ -1,5 +1,5 @@
 Package: parallelly
-Version: 1.46.0-9006
+Version: 1.46.0-9007
 Title: Enhancing the 'parallel' Package
 Imports:
     parallel,

NEWS.md

@@ -2,6 +2,11 @@
 
 ## Bug Fixes
 
+ * Loading the package in the Positron Console would incorrectly set
+   environment variable `_R_CHECK_LIMIT_CORES_` to TRUE, which in turn
+   would result in `availableCores()` being limited to a maximum of
+   two (2) CPU cores.
+
  * `parallel::clusterExport()` on a `makeClusterSequential()` cluster
    would export to the global environment rather than the local
    environment of the cluster nodes.

R/utils.R

@@ -114,11 +114,11 @@ inRCmdCheck <- function() { queryRCmdCheck() != "notRunning" }
 
 inRCmdVignette <- function() {
   r_cmd <- Sys.getenv("R_CMD", NA_character_)
-  if (is.null(r_cmd)) return(FALSE)
+  if (is.na(r_cmd)) return(FALSE)
   args <- commandArgs()
   if (length(args) != 6) return(FALSE)
   truth <- c(r_cmd, "--no-echo", "--no-restore", "--vanilla", "-e", "tools::buildVignettes(dir~+~=~+~'.',~+~tangle~+~=~+~TRUE)")
-  any(args != truth)
+  any(args == truth)
 }
 
 commaq <- function(x, sep = ", ") paste(sQuote(x), collapse = sep)