Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add transitive package dependency discovery support for ‘renv’ #368

Open
klmr opened this issue Jun 29, 2024 · 0 comments
Open

Add transitive package dependency discovery support for ‘renv’ #368

klmr opened this issue Jun 29, 2024 · 0 comments

Comments

@klmr
Copy link
Owner

klmr commented Jun 29, 2024

Please describe your feature request

‘renv’ already supports box::use() declarations for dependency discovery via static analysis.

However, this misses all package dependencies declared inside modules that are imported via box::use(), in particular non-local modules (local modules would presumably be searched by ‘renv’ anyway). Adding support for the discovery of transitive package dependencies declared inside modules used by a project to ‘renv’ should be feasible, but would require more complex static analysis, in combination with a reimplementation of the module finding algorithm used by ‘box’.

A better solution would be for ‘box’ to expose supporting infrastructure that could be invoked by ‘renv’. This is kind of “safe”, since a ‘renv’ project that uses ‘box’ presumably has ‘box’ installed anyway, so ‘renv’ could soft-depend on it (via Enhances; or it might not even need that?).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant