Skip to content

Added an new item how to link intel statically #7

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

Merged
merged 4 commits into from
Oct 19, 2015

Conversation

anhzhang
Copy link
Contributor

This is the fix to #818, open-mpi/ompi#818


$anchor[] = "statically-link-with-intel-libraries";

$a[] = "When compiled with Intel compiler suite, by default Openmpi binaries are
Copy link
Member

Choose a reason for hiding this comment

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

"Open MPI", not "Openmpi".

I'd change the first sentence to:

The Intel compiler suite, by default, dynamically links its runtime libraries against the Open MPI binaries and libraries. This can cause problems if the Intel compiler libraries are installed in non-standard locations. For example, you might get errors like:

...put verbatim error messages here...

To avoid such problems, you can pass flags to Open MPI's configure script that instruct the Intel compiler suite to statically link its runtime libraries with Open MPI:

...geshi bash configure example...

Improved it based on Jeff's nice review comments.
@anhzhang
Copy link
Contributor Author

@jsquyres Thank you very much :)
Today I cloned the ompi-www to my desktop, it is very large, our network is slow, anyway, I got it and installed the nginx and php, found some minor format issue, I will have an new update tomorrow, thanks.

@jsquyres
Copy link
Member

Yeah, it is large -- sorry about that. Thanks for the update.

@anhzhang
Copy link
Contributor Author

Hi, @jsquyres I have tested my changes with nginx/php5, it is good now, please review it again, thanks a lot!

@jsquyres
Copy link
Member

Looks good. Thanks!

jsquyres added a commit that referenced this pull request Oct 19, 2015
Added an new item how to link intel statically
@jsquyres jsquyres merged commit 16d8dbb into open-mpi:master Oct 19, 2015
bwbarrett pushed a commit that referenced this pull request Oct 7, 2018
Added an new item how to link intel statically

Former-commit-id: 16d8dbb
bwbarrett pushed a commit that referenced this pull request Oct 7, 2018
Added an new item how to link intel statically

Former-commit-id: 16d8dbb
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