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

update $R_LIBS_SITE rather than $R_LIBS in easyconfigs installing R packages #12534

Merged
merged 7 commits into from
Apr 6, 2021

Conversation

boegel
Copy link
Member

@boegel boegel commented Apr 6, 2021

@boegel
Copy link
Member Author

boegel commented Apr 6, 2021

Hmm, I'm not sure it's worth adding the missing checksums for those ancient easyconfigs (but I do want to change the R_LIBS to R_LIBS_SITE there)...

Full list of touched easyconfigs with missing checksums:

OOMPA-3.1.2-intel-2016b-R-3.3.1.eb
R-bundle-Bioconductor-3.2-foss-2016a-R-3.2.3.eb
R-bundle-Bioconductor-3.2-intel-2016a-R-3.2.3.eb
R-bundle-Bioconductor-3.3-intel-2016b-R-3.3.1.eb
R-bundle-Bioconductor-3.5-intel-2017a-R-3.4.0.eb
evmix-2.6-intel-2016b-R-3.3.1.eb
spectral.methods-0.7.2.133-intel-2017b-R-3.4.3.eb
spectral.methods-0.7.2.133-intel-2018a-R-3.4.4.eb

@boegel
Copy link
Member Author

boegel commented Apr 6, 2021

Test report by @boegel
SUCCESS
Build succeeded for 30 out of 30 (30 easyconfigs in total)
node2700.swalot.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/ff1a7f2224590fba0b1f52c127b68150 for a full test report.

@easybuilders easybuilders deleted a comment from boegelbot Apr 6, 2021
@boegel
Copy link
Member Author

boegel commented Apr 6, 2021

Test report by @boegel
SUCCESS
Build succeeded for 4 out of 4 (4 easyconfigs in total)
node2700.swalot.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/e5ad1ffc882ebb23cc723e1ffcc432e1 for a full test report.

@easybuilders easybuilders deleted a comment from boegelbot Apr 6, 2021
Copy link
Member

@branfosj branfosj left a comment

Choose a reason for hiding this comment

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

lgtm

@branfosj
Copy link
Member

branfosj commented Apr 6, 2021

Going in, thanks @boegel!

@branfosj branfosj merged commit 9821503 into easybuilders:develop Apr 6, 2021
@boegel boegel deleted the R_LIBS_SITE branch April 6, 2021 20:12
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