Skip to content

Conversation

@gadenbuie
Copy link
Member

@gadenbuie gadenbuie commented Jul 22, 2022

This PR fixes #331 by having copyDependencyToDir() create the outputDir recursively. Previously, this was a side effect of a later call to file.copy(), so this change makes the recursive directory creation explicit. It's also necessary when a nested directory is required because normalizePath() will fail otherwise.

@gadenbuie
Copy link
Member Author

cc @cpsievert @schloerke since I can't request reviewers in this repo.

@cpsievert
Copy link
Collaborator

Thank you!

@cpsievert cpsievert merged commit 95f2796 into rstudio:main Oct 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

copyDependencyToDir() should create outputDir recursively

2 participants