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 butcher package (version 0.1.5 is the latest that can be achieved - need to add [email protected] first) #127

Open
remlapmot opened this issue Feb 19, 2023 · 0 comments

Comments

@remlapmot
Copy link
Contributor

For completeness, given that butcher is mentioned in the docs here I had a look at adding it using the new machinery.

Previously, using snapshot repositories I had worked out that I could install butcher version 0.1.5 without updating any of its dependencies, and this also added and 1 new dependency package, lobstr.

A bit to my surprise naively running

just add-package [email protected]

not only added lobstr but bumped cpp11 and rlang. This turned out to because lobstr 1.1.2 was added because that is its current version on CRAN, and that requires the bumps to the other 2. Whereas the snapshot date I used had version 1.1.1 as its current version of lobstr.

Hence using the new machinery, running in order

just add-package [email protected]
just add-package [email protected]

achieves the installation of butcher without bumping the other 2.

@bloodearnest - this made me think - do you think you would allow adding an additional repos parameter to the add-package recipe, because I could achieve those 2 lines above with a single line with syntax such as

just add-package [email protected] https://packagemanager.posit.co/cran/2021-06-29

where the new repos parameter is passed to the renv::install() call on line 69 of the Dockerfile, i.e., that syntax would expand to

renv::install("[email protected]", repos = "https://packagemanager.posit.co/cran/2021-06-29")

Having this parameter would make it easier to add packages without bumping dependencies in the future (because otherwise you have to manually work out the order in which to add the new dependencies, which could be tricky if there was more than 1 as in this case).

inglesp added a commit that referenced this issue Feb 23, 2023
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

No branches or pull requests

1 participant