Skip to content

Conversation

@rhc54
Copy link
Contributor

@rhc54 rhc54 commented Sep 8, 2015

No description provided.

@jsquyres
Copy link
Member

jsquyres commented Sep 8, 2015

@rhc54 Do the wrappers really need the -I's for pmix?

@rhc54
Copy link
Contributor Author

rhc54 commented Sep 8, 2015

@jsquyres When I run the whitespace purge, it hits about a few (small number) files that this PR doesn't touch. Want me to just include everything here?

@jsquyres jsquyres changed the title Add the libs required for PMIx to support static builds Add the libs required for PMIx to support static builds (and trim all excess whitespace) Sep 8, 2015
@jsquyres
Copy link
Member

jsquyres commented Sep 8, 2015

I edited the title, and now it says that that is exactly what you are supposed to do. 😄

rhc54 pushed a commit that referenced this pull request Sep 8, 2015
Add the libs required for PMIx to support static builds (and trim all excess whitespace)
@rhc54 rhc54 merged commit 8053357 into open-mpi:master Sep 8, 2015
@rhc54 rhc54 deleted the topic/static branch November 3, 2015 15:49
jsquyres pushed a commit to jsquyres/ompi that referenced this pull request Sep 19, 2016
…_missing_include_file

pml/cm: add missing #include <alloca.h>
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.

2 participants