rstudio / rstudio/packrat

packrat::install() ignores "dependencies" parameter

Open
#413 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
R
Stars
409
Forks
89
PR merge metrics
No merged PRs in 30d

Description

I was having some trouble getting the "Suggests" dependencies of one of my local packages to install. The docs suggested that I should be able to add a "dependencies" param to packrat::install_local(), and it would be passed through to packrat::install(), where it would behave the same as in utils::install.packages().

It didn't work, so I took a look at the source code, and as far as I can tell, the "dependencies" parameter is declared (in R/install.R, on line 43, as part of the function definition for install), but then it's never referenced. And there doesn't seem to be any code that passes them on as, say, part of the whole argument list.

If I'm reading the git history correctly, it looks like this was a result of commit f9038b8c, which replaced a dependency on devtools::install() with packrat's own install() function.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading R/install.R around install() and tracing the install_local() call path; use utils::install.packages() as the behavioral reference. Done means the dependencies argument is honored so local packages' Suggests dependencies install as documented.

Written by the indexing model from the issue text.

Assessment

Tech stack
r
Domain
tooling
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.