Skip to content

Commit

Permalink
WORKAROUND: unzip may be set to "" on some Linux systems, which cause (
Browse files Browse the repository at this point in the history
…#57)

remotes::install_remote() to generate an obscure error message:

Error in system_check(unzip, args) :
  Command  failed sh: : command not found

Since "internal" unzip always, work this patch makes sure that is
used also when unzip is "".

See https://bugs.r-project.org/bugzilla/show_bug.cgi?id=17255 for
more details on why unzip may become "".
  • Loading branch information
HenrikBengtsson authored and gaborcsardi committed Apr 8, 2017
1 parent e1eb768 commit ba7a8bc
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion R/decompress.R
Expand Up @@ -75,7 +75,7 @@ getrootdir <- function(file_list) {
}

my_unzip <- function(src, target, unzip = getOption("unzip")) {
if (unzip == "internal") {
if (unzip %in% c("internal", "")) {
return(utils::unzip(src, exdir = target))
}

Expand Down

0 comments on commit ba7a8bc

Please sign in to comment.