BUG FIX: future(..., packages = "missing-package") did result in an error despite requesting a non-installed package

Author: HenrikBengtsson

DESCRIPTION

@@ -1,5 +1,5 @@
 Package: future
-Version: 1.68.0-9014
+Version: 1.68.0-9015
 Title: Unified Parallel and Distributed Processing in R for Everyone
 Depends:
     R (>= 3.2.0)

NEWS.md

@@ -17,6 +17,9 @@
    1 unknown future arguments: 'interrupts'" for third-party future
    backends.
 
+ * `future(..., packages = "missing-package")` did result in an error
+   despite requesting a non-installed package.
+
 ## Deprecated and Defunct
 
  * Remove argument `earlySignal` from `future()`, `futureAssign()`,

R/backend_api-evalFuture.R

@@ -667,9 +667,12 @@ evalFutureInternal <- function(data) {
   ## -----------------------------------------------------------------
   ## Load and attached backend packages
   ## -----------------------------------------------------------------
-  withCallingHandlers({
+  res <- withCallingHandlers({
     attachPackages(backendPackages)
   }, condition = onEvalCondition)
+  if (inherits(res, "error")) {
+    stop(FutureEvalError(sprintf("Failed to attach one or more future-backend packages: %s", conditionMessage(res))))
+  }
 
 
   ## -----------------------------------------------------------------
@@ -682,9 +685,12 @@ evalFutureInternal <- function(data) {
   ...future.mc.cores.old <- getOption("mc.cores")
 
   ## Load and attached packages
-  withCallingHandlers({
+  res <- withCallingHandlers({
     attachPackages(packages)
   }, condition = onEvalCondition)
+  if (inherits(res, "error")) {
+    stop(FutureEvalError(sprintf("Failed to attach one or more packages: %s", conditionMessage(res))))
+  }
 
   ## Note, we record R options and environment variables _after_
   ## loading and attaching packages, in case they set options/env vars