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.

Sign up for GitHub

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

Jump to bottom

{toolchain}[dummy/dummy] goolfc v2017b #5768

Merged
merged 2 commits into from
Feb 13, 2018
Merged

{toolchain}[dummy/dummy] goolfc v2017b #5768

merged 2 commits into from
Feb 13, 2018

Conversation

akesandgren
Copy link
Contributor

(created using eb --new-pr)
New version of goolfc toolchain. Uses same versions of dependencies as foss/2017b but adds CUDA 9.0

@akesandgren akesandgren mentioned this pull request Feb 12, 2018
Merged
boegel
boegel requested changes Feb 12, 2018
View reviewed changes
easybuild/easyconfigs/c/CUDA/CUDA-9.0.176-GCC-6.4.0-2.28.eb Outdated
'https://developer.nvidia.com/compute/%(namelower)s/%(version_major_minor)s/Prod/local_installers/',
]
sources = ['%%(namelower)s_%%(version)s_%s_linux-run' % nv_version]
#checksums = ['8496c72b16fee61889f9281449b5d633d0b358b46579175c275d85c9205fe953']
Copy link
Member

Choose a reason for hiding this comment

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

@akesandgren Please fix? I have 96863423feaa50b5c1c5e1b9ec537ef7ba77576a3986652351ae43e66bcd080c as SHA256 for cuda_9.0.176_384.81_linux-run, this one you have here is the one for cuda_9.1.85_387.26_linux

Copy link
Contributor Author

@akesandgren akesandgren Feb 12, 2018 •
edited
Loading

Choose a reason for hiding this comment

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

Oops, must have forgotten to send that update.

@boegel boegel added this to the next release (3.5.2 or 3.6.0) milestone Feb 12, 2018
@boegel boegel added the update label Feb 12, 2018
@easybuilders easybuilders deleted a comment from boegelbot Feb 12, 2018
@boegel
Copy link
Member

boegel commented Feb 12, 2018

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

@boegel
Copy link
Member

boegel commented Feb 13, 2018

Test report by @boegel
SUCCESS
Build succeeded for 7 out of 7 (7 easyconfigs in this PR)
node2567.golett.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/674321b1804a0b39f4791abdff153f40 for a full test report.

@boegel
Copy link
Member

boegel commented Feb 13, 2018

Going in, thanks @akesandgren!

@boegel boegel merged commit 4121a6c into easybuilders:develop Feb 13, 2018
@akesandgren akesandgren deleted the 20180202171349_new_pr_goolfc2017b branch February 13, 2018 18:36
@wpoely86
Copy link
Member

wpoely86 commented Feb 14, 2018 •
edited
Loading 

libtool:   error: require no space between '-L' and '-lutil'

That's new for me (in the build of openmpi)

@wpoely86
Copy link
Member

In openmpi-2.1.1/ompi/mca/pml/ucx/Makefile I see:

atomic_ucx_LIBS =  -lucp -L

Not sure what is supposed to be there.

@akesandgren
Copy link
Contributor Author

Odd, build with --debug and check what the configure step says.

@akesandgren
Copy link
Contributor Author

akesandgren commented Feb 14, 2018 •
edited
Loading

I have this on my laptop (i.e. no infiniband for instance)
atomic_ucx_LIBS =
so you'll have to figure out where it comes from

@boegel
Copy link
Member

boegel commented Feb 21, 2018

@wpoely86 Have you figured this out yet?

@akesandgren
Copy link
Contributor Author

Ah, it's the UCX framework? Is that installed as a OS package?
If so there might be a bug in OpenMPI if it shows just -L

@wpoely86
Copy link
Member

Yes, removing the rpms for UCX fixed it. It seems the configure script fails to do it correctly.

@boegel boegel mentioned this pull request Feb 23, 2018
Closed
@boegel
Copy link
Member

boegel commented Feb 23, 2018

@wpoely86 See #5805?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@boegel boegel boegel approved these changes

Assignees
No one assigned
Labels
update
Projects
None yet
Milestone
3.5.2
Development

Successfully merging this pull request may close these issues.
3 participants
@akesandgren @boegel @wpoely86