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

{toolchain}[dummy/dummy] goolfc v2018a #6129

Merged

Conversation

akesandgren
Copy link
Contributor

(created using eb --new-pr)
New goolfc version 2018a. Based on foss/2018a + CUDA 9.1

@vanzod
Copy link
Member

vanzod commented Apr 6, 2018

@akesandgren If this is alligned with foss, didn't @boegel agreed to rename the toolchain fosscuda?
I know I promised to work on this but I have not had time yet...

@akesandgren
Copy link
Contributor Author

I don't know. I missed the last couple of meetings (and forgot to read the notes).
But since our users are accustomed to using goolfc for this I'll keep it up until told otherwise :-)

@boegel
Copy link
Member

boegel commented Apr 7, 2018

Rebranding goolfc as fosscuda makes sense to me...

@boegel boegel added the update label Apr 7, 2018
@boegel boegel added this to the 3.6.0 milestone Apr 7, 2018
@boegel
Copy link
Member

boegel commented Apr 7, 2018

Test report by @boegel
SUCCESS
Build succeeded for 6 out of 6 (6 easyconfigs in this PR)
node2113.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/0039bd4e83ba603dbfdce64de40fc7dc for a full test report.

@boegel
Copy link
Member

boegel commented Apr 7, 2018

Test report by @boegel
SUCCESS
Build succeeded for 7 out of 7 (6 easyconfigs in this PR)
node2637.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/5a0679ec8c60e295126c1c1e2efb6e01 for a full test report.

Copy link
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
Member

boegel commented Apr 7, 2018

I propose we add fosscuda equivalents for the "common" goolfc versions in a separate PR, by means of transition...

@boegel
Copy link
Member

boegel commented Apr 7, 2018

Going in, thanks @akesandgren!

@boegel boegel merged commit d0e4957 into easybuilders:develop Apr 7, 2018
@akesandgren akesandgren deleted the 20180406214636_new_pr_goolfc2018a branch July 5, 2018 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants