-
Notifications
You must be signed in to change notification settings - Fork 714
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
{lang}[GCC/13.2.0,GCCcore/13.2.0,gfbf/2023b] SOCI v4.0.3, yaml-cpp v0.8.0, RStudio-Server v2023.12.1+402 w/ Java 11 R 4.3.3 #20060
{lang}[GCC/13.2.0,GCCcore/13.2.0,gfbf/2023b] SOCI v4.0.3, yaml-cpp v0.8.0, RStudio-Server v2023.12.1+402 w/ Java 11 R 4.3.3 #20060
Conversation
…13.2.0.eb, RStudio-Server-2023.12.1+402-gfbf-2023b-Java-11-R-4.3.3.eb
Test report by @lazariv |
...uild/easyconfigs/r/RStudio-Server/RStudio-Server-2023.12.1+402-gfbf-2023b-Java-11-R-4.3.3.eb
Outdated
Show resolved
Hide resolved
...uild/easyconfigs/r/RStudio-Server/RStudio-Server-2023.12.1+402-gfbf-2023b-Java-11-R-4.3.3.eb
Outdated
Show resolved
Hide resolved
I suspect you have parallel installations there as the error complains about missing package "brio" in R as for combining ECs for PRs: It is usually better to open individual/smaller PRs. You did that for R itself, so don't include that EC here in this PR. Same for libgit2 which is a dependency of R and also in that other PR. Having them in both PR results in needless and potentially conflicting rebuilds. |
Test report by @lazariv |
...uild/easyconfigs/r/RStudio-Server/RStudio-Server-2023.12.1+402-gfbf-2023b-Java-11-R-4.3.3.eb
Outdated
Show resolved
Hide resolved
easybuild/easyconfigs/y/yaml-cpp/yaml-cpp-0.8.0-GCCcore-13.2.0.eb
Outdated
Show resolved
Hide resolved
…1+402-gfbf-2023b-Java-11-R-4.3.3.eb Co-authored-by: Alexander Grund <[email protected]>
…1+402-gfbf-2023b-Java-11-R-4.3.3.eb Co-authored-by: Jasper Grimm <[email protected]>
…asyconfigs into 20240307090339_new_pr_SOCI403
Removed unnecessary patches from 0.7.0
Test report by @lazariv |
Test report by @lazariv |
Only Boost/1.83.0 and PostgreSQL/16.1 is missing. Rebuilding with |
Test report by @lazariv |
Test report by @lazariv |
Test report by @verdurin |
@boegelbot please test @ generoso |
@verdurin: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1994339684 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
…asyconfigs into 20240307090339_new_pr_SOCI403
@jfgrimm anything should be done here? |
@lazariv no, this looks ready to be merged. I'll send the last remaining mandatory test. |
…asyconfigs into 20240307090339_new_pr_SOCI403
@boegelbot: please test @ generoso |
@lexming: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2133791595 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@boegelbot please test @ jsc-zen3 |
@lexming: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2134059566 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@boegelbot please test @ jsc-zen3 |
@lexming: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2134070963 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems that in the meantime the tags of the rstudio repo gained the typical v
prefix. No idea when that happened, but I had to update the tarball name of the sources.
LGTM
Merging, thanks @lazariv ! |
(created using
eb --new-pr
)Changes compared to RStudio-Server-2022.07.2%2B576-foss-2022a-Java-11-R-4.2.1.eb:
RStudio-Server-2022.07.1+554_allow-disabling-quarto.patch
not required, fixed in rstudio/rstudio@9b7ac4bRStudio-Server-2022.07.1+554_fix-libsoci-search.patch
fixed with Allow to find system SOCI_* libraries rstudio/rstudio#11691RStudio-Server-2022.07.1+554_use-XDG_CACHE_HOME.patch
fixed with Use XDG_CACHE_HOME for secure cookie key in low privilege mode rstudio/rstudio#11976('nodejs', '20.9.0')
required for building: https://github.com/rstudio/rstudio/blob/4da58325ffcff29d157d9264087d4b1ab27f7204/src/gwt/build.xml#L142