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

Merge dev to main #770

Merged
merged 7 commits into from
May 14, 2024
Merged

Merge dev to main #770

merged 7 commits into from
May 14, 2024

Conversation

ianpittwood
Copy link
Collaborator

No description provided.

Lytol and others added 7 commits May 7, 2024 10:32
* Add `--no-install-recommends` to Ubuntu shared scripts

- Prevent `r-recommended` from being unintentionally installed
- Stop installing `r-base-core` since it adds an R installation unnecessarily
- Check that `r-recommended` and system R version are not present
- Add `--no-install-recommends` to all shared scripts, it's best practice for images

* Remove duplicated variable

* Remove install_r.sh dependency installation

Add common R dependencies to ubuntu2204_packages.txt
* Bump r version to 4.4 to address CVE and alt version to 4.3.3

* Fix the bake files

* Update r version and alt version in Workbench image

* Third time's the charm.

* Remove duplicates

---------

Co-authored-by: Ian Pittwood <ian.pittwood@posit.co>
* Set the bakefile dynamically for manual builds

* Echo bake file

* Add needs to build job

* Dynamically set product version

* Set shell

* Fix else if

* Set versions to empty string

* Dynamically set target name

* Set all versions to literal "null" strings, they'll fail anyways this way

* Temporarily add set-bake-file-in-manual as usable branch

* Remove concurrency restriction for manual builds

* Remove literal null strings

* Fix get-version script including "" wrap on versions

* Remove content-images

* Remove testing lines

Add future debugging lines
@ianpittwood ianpittwood merged commit 7a02449 into main May 14, 2024
1030 of 1040 checks passed
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.

4 participants