Skip to content

Commit c24c174

Browse files
authored
Fixes out of bounds error (#1717)
* Now handles single result from system2 * comments out env installation prompt
1 parent ecf2061 commit c24c174

File tree

2 files changed

+16
-10
lines changed

2 files changed

+16
-10
lines changed

R/config.R

+9-8
Original file line numberDiff line numberDiff line change
@@ -492,14 +492,15 @@ try_create_default_virtualenv <- function(package = "reticulate", ...) {
492492
return(NULL)
493493

494494
if (permission == "") {
495-
if (is_interactive()) {
496-
permission <- utils::askYesNo(sprintf(
497-
"Would you like to create a default Python environment for the %s package?",
498-
package))
499-
if (!isTRUE(permission))
500-
return(NULL)
501-
permission <- "true"
502-
}
495+
return(NULL)
496+
# if (is_interactive()) {
497+
# permission <- utils::askYesNo(sprintf(
498+
# "Would you like to create a default Python environment for the %s package?",
499+
# package))
500+
# if (!isTRUE(permission))
501+
# return(NULL)
502+
# permission <- "true"
503+
# }
503504
}
504505

505506
if (!permission %in% c("true", "yes", "1"))

R/py_require.R

+7-2
Original file line numberDiff line numberDiff line change
@@ -488,8 +488,13 @@ uv_get_or_create_env <- function(packages = py_reqs_get("packages"),
488488
if (cmd_failed) {
489489
cmd_err <- paste0(result, collapse = "\n")
490490
} else {
491-
cmd_err <- paste0(result[[1]], "\n")
492-
cmd_out <- result[[2]]
491+
if (length(result) == 1) {
492+
cmd_out <- result[[1]]
493+
cmd_err <- NULL
494+
} else {
495+
cmd_out <- result[[2]]
496+
cmd_err <- paste0(result[[1]], "\n")
497+
}
493498
}
494499
}
495500

0 commit comments

Comments
 (0)