renv.config.copy.method

harrismcgehee · May 15, 2024, 12:36pm

Can anyone help me figure out if the renv.config.copy.method configuration option is case-sensitive?

Does "R" == "r" for this setting?

Thanks!

randyzwitch · May 15, 2024, 6:55pm

In the documentation, it specifically has it as uppercase R (as opposed to others that are lowercase), so I would specify it as uppercase

github.com

rstudio/renv/blob/5d0d52c395e569f7f24df4288d949cef95efca4e/man/config.Rd#L153


      
          default, renv will try to choose a system tool that is likely to succeed in
          copying files on your system -- \code{robocopy} on Windows, and \code{cp} on Unix.
          renv will also instruct these tools to preserve timestamps and attributes
          when copying files. However, you can select a different method as
          appropriate.
          
          The following methods are supported:
          
          \tabular{ll}{
          \code{auto}     \tab Use \code{robocopy} on Windows, and \code{cp} on Unix-alikes. \cr
          \code{R}        \tab Use \R's built-in \code{file.copy()} function. \cr
          \code{cp}       \tab Use \code{cp} to copy files. \cr
          \code{robocopy} \tab Use \code{robocopy} to copy files. (Only available on Windows.) \cr
          \code{rsync}    \tab Use \code{rsync} to copy files. \cr
          }
          
          You can also provide a custom copy method if required; e.g.
          
          \if{html}{\out{<div class="sourceCode">}}\preformatted{options(renv.config.copy.method = function(src, dst) \{
            # copy a file from 'src' to 'dst'
          \})

Best,
Randy

harrismcgehee · May 15, 2024, 7:11pm

Do you have any insight into how it is converted from the "R" in the documentation to the "r" in the code?

github.com

rstudio/renv/blob/5d0d52c395e569f7f24df4288d949cef95efca4e/R/files.R#L125


      
            status <- catchall(renv_file_move(tempfile, target))
            if (inherits(status, "condition"))
              stop(status)
          
          }
          
          renv_file_copy_dir_impl <- function(source, target) {
          
            methods <- list(
              cp       = renv_file_copy_dir_cp,
              r        = renv_file_copy_dir_r,
              robocopy = renv_file_copy_dir_robocopy,
              rsync    = renv_file_copy_dir_rsync
            )
          
            copy <- config$copy.method()
            if (is.function(copy))
              return(copy(source, target))
          
            method <- methods[[tolower(copy)]]
            if (!is.null(method))

harrismcgehee · May 15, 2024, 7:11pm

Oh! I see it @ 134!

method <- methods[[tolower(copy)]]

case insensitive!

system · May 22, 2024, 7:12pm

This topic was automatically closed 7 days after the last reply. New replies are no longer allowed.

If you have a query related to it or one of the replies, start a new topic and refer back with a link.