pak icon indicating copy to clipboard operation
pak copied to clipboard

move sudo to outside `execute` of `system_requirements_internal`

Open lz100 opened this issue 3 years ago • 0 comments

If we just want to print out the command with sudo but does not want to execute the command, current 'pak' does not prepend "sudo" unless you combine execute = TRUE, e.g

pak::pkg_system_requirements("openssl", sudo = TRUE, execute = FALSE)
##  [1] "apt-get install -y libssl-dev"

This pull request moves sudo outside execute, so when sudo = TRUE, execute = FALSE, the sudo will still be prepend to output.

lz100 avatar Mar 02 '21 22:03 lz100