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

--with-mpi #1

Closed
bjoo opened this issue Feb 3, 2011 · 4 comments
Closed

--with-mpi #1

bjoo opened this issue Feb 3, 2011 · 4 comments

Comments

@bjoo
Copy link
Member

bjoo commented Feb 3, 2011

Hi Guys,
--with-mpi on the configure script appears to be overloaded. I originally specified it to point to the location of the MPI distro which was used to compile QMP programs. It has now been overloaded to also define -DUSE_MPI (to select MPI vs QMP comms harness). This is bad because it is natural to have --with-qmp= --with-mpi= set together for a normal QMP build, and this will add -DQMP_COMMS and -DMPI_COMMS on the command line.

We should find a way around. How about

--with-qmp= sets -DQMP_COMMS
--with-mpi= sets -DMPI_COMMS
--with-mpihome= sets MPI_HOME, but not -DMPI_COMMS

Thoughts?

@gshi
Copy link
Member

gshi commented Feb 3, 2011

Sounds good to me. Please go ahead and implement it.

@bjoo
Copy link
Member Author

bjoo commented Feb 4, 2011

RESOLVED: Now with --with-mpi= on its own means "define -DMPI_COMMS and build for MPI using the MPI in ".

--with-qmp= together with --with-mpi= means: "define -DQMP_COMMS and build for QMP, but using the mpicc, mpirun etc in "

@gshi
Copy link
Member

gshi commented Feb 4, 2011

That's more intuitive, better solution. Thanks

@rbabich
Copy link
Member

rbabich commented Feb 5, 2011

Agreed. I like this.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants