Skip to content

Change 'rstudio' name to 'RStudio-Server' to agree with official name and better reflect what it provides#11764

Merged
boegel merged 3 commits intoeasybuilders:developfrom
ocaisa:RStudio
Nov 26, 2020
Merged

Change 'rstudio' name to 'RStudio-Server' to agree with official name and better reflect what it provides#11764
boegel merged 3 commits intoeasybuilders:developfrom
ocaisa:RStudio

Conversation

@ocaisa
Copy link
Copy Markdown
Member

@ocaisa ocaisa commented Nov 25, 2020

This more accurately reflects what is actually being installed

@ocaisa
Copy link
Copy Markdown
Member Author

ocaisa commented Nov 25, 2020

Test reports on the way

@ocaisa
Copy link
Copy Markdown
Member Author

ocaisa commented Nov 25, 2020

Test report by @ocaisa
SUCCESS
Build succeeded for 3 out of 3 (3 easyconfigs in total)
generoso - Linux centos linux 8.2.2004, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz, Python 3.6.8
See https://gist.github.com/7ab7df101a3926b44f06f637aea3ff3f for a full test report.

@ocaisa
Copy link
Copy Markdown
Member Author

ocaisa commented Nov 26, 2020

I can't do the test for fosscuda because R requires libcuda.so to be available during the build (and it's not on generoso) but the changes there are identical to the foss version anyway

@boegel boegel added this to the 4.3.2 (next release) milestone Nov 26, 2020
@boegel boegel changed the title Correct rstudio name to RStudio-Server Change 'rstudio' name to 'RStudio-Server' to agree with official name and better reflect what it provides Nov 26, 2020
@boegel
Copy link
Copy Markdown
Member

boegel commented Nov 26, 2020

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3307.joltik.os - Linux centos linux 7.8.2003, x86_64, Intel(R) Xeon(R) Gold 6242 CPU @ 2.80GHz (cascadelake), Python 3.6.8
See https://gist.github.com/00afb8dccff41df3429d61e156d3b27c for a full test report.

Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@boegel
Copy link
Copy Markdown
Member

boegel commented Nov 26, 2020

Going in, thanks @ocaisa!

@boegel boegel merged commit c9a7234 into easybuilders:develop Nov 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants