Skip to content

clean up configure options that are now handled by R easyblock#5478

Merged
migueldiascosta merged 1 commit intoeasybuilders:developfrom
boegel:R_cleanup
Dec 8, 2017
Merged

clean up configure options that are now handled by R easyblock#5478
migueldiascosta merged 1 commit intoeasybuilders:developfrom
boegel:R_cleanup

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Dec 6, 2017

@boegel boegel added this to the 3.5.0 milestone Dec 6, 2017
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Dec 6, 2017

I'll look into a test report for several recent and also a couple of old R easyconfigs, but a full test is going to take forever (and probably not worth the trouble, since nothing really changed as long as you use the updated easyblock for R)...

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Dec 7, 2017

Test report by @boegel
SUCCESS
Build succeeded for 54 out of 54 (5 easyconfigs in this PR)
node2644.swalot.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/2a40f275f8fdfcce6ea5e2a173215557 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Dec 7, 2017

Test report by @boegel
SUCCESS
Build succeeded for 10 out of 10 (10 easyconfigs in this PR)
node2038.delcatty.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/a3bcca289d3ddf83ebb26674fab906d5 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Dec 7, 2017

@easybuilders/easybuild-easyconfigs-maintainers Good to go imho...

Copy link
Copy Markdown
Member

@migueldiascosta migueldiascosta left a comment

Choose a reason for hiding this comment

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

lgtm

@migueldiascosta
Copy link
Copy Markdown
Member

Going in, thanks @boegel!

@migueldiascosta migueldiascosta merged commit 0ce94b9 into easybuilders:develop Dec 8, 2017
@boegel boegel deleted the R_cleanup branch December 8, 2017 07:55
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