Skip to content

Conversation

@karasevb
Copy link
Member

Adds detecting UCX from following paths: "/usr /usr/local /opt/ucx"

Signed-off-by: Boris Karasev karasev.b@gmail.com

Adds detecting UCX from following paths: "/usr /usr/local /opt/ucx"

Signed-off-by: Boris Karasev <karasev.b@gmail.com>
@karasevb karasevb requested review from artpol84 and rhc54 July 25, 2017 20:51
@rhc54 rhc54 removed their request for review July 25, 2017 21:39
@rhc54
Copy link
Contributor

rhc54 commented Jul 25, 2017

I can't really help you here as I don't have UCX on my systems

@jladd-mlnx jladd-mlnx requested a review from hppritcha July 25, 2017 22:19
@jladd-mlnx
Copy link
Member

@hppritcha can you have your student give this a spin, please?

@ibm-ompi
Copy link

The IBM CI (GNU Compiler) build failed! Please review the log, linked below.

Gist: https://gist.github.com/f0f114ad6092fea29ce473c02aa8619a

@ibm-ompi
Copy link

The IBM CI (XL Compiler) build failed! Please review the log, linked below.

Gist: https://gist.github.com/eff8f5836006d86c583b8b8f82dac95c

@ibm-ompi
Copy link

The IBM CI (PGI Compiler) build failed! Please review the log, linked below.

Gist: https://gist.github.com/2e251aec57361b78d479c7b5e0ff243f

Signed-off-by: Boris Karasev <karasev.b@gmail.com>
@artpol84
Copy link
Contributor

@hppritcha do you know if you'll be able to verify?

@hppritcha
Copy link
Member

works for me (using pkg-config)

@jladd-mlnx jladd-mlnx merged commit c27beea into open-mpi:master Aug 3, 2017
@artpol84
Copy link
Contributor

artpol84 commented Aug 3, 2017

@karasevb, please cherry-pick to v3.0.x

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants