-
Notifications
You must be signed in to change notification settings - Fork 66
{2023.06}[2023a,2023b] RStudio-Server 2023.09.1+494, RStudio-Server 2024.09.0+375 #1194
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
base: main
Are you sure you want to change the base?
Conversation
PetrKralCZ
commented
Sep 18, 2025
|
I'm very wary of the fact that the recipe for Compute Canada for RStudio is not the same as the upstream recipe, in particular the patch https://github.com/ComputeCanada/easybuild-easyconfigs-installed-avx2/blob/main/2023/RStudio-Server/rstudio-1.2.1335.patch . I think it is worth exploring the differences before considering merging this. |
|
@ccoulombe I used to know why that patch was necessary but I can't fully remember, can you help explain? I think it is needed so that you are able to use different R versions in the same session. |
I see that patch is no longer applied to their latest builds so it may no longer be needed to worry about it. I'm willing to build and include it so we can actually test it out, if we need to rebuild later then so be it. |
|
bot: build repo:eessi.io-2023.06-software instance:eessi-bot-mc-aws for:arch=x86_64/amd/zen2 |
|
New job on instance
|
|
New job on instance
|
|
I don't see too much value in adding an old version so I will stick with supporting the later version and open a seperate PR |
|
Splitting off the newer version into #1292 |