Ticket #4961 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Allow use of alternative MPI compilers

Reported by: Russell Taylor Owned by: Russell Taylor
Priority: trivial Milestone: Release 2.1
Component: Mantid Keywords: MPI
Cc: Blocked By:
Blocking: Tester: Stuart Campbell

Description

Right now, the name of the MPI compiler is hard-coded into the CMakeLists (as mpicc/mpicxx). The FindMPI.cmake module defines the variables MPI_CXX_COMPILER & MPI_C_COMPILER but these are presently ignored. Change to picking up the compiler names from these (which will default to the same as at present) so that someone can change the compiler used via, e.g., ccmake.

Change History

comment:1 Changed 9 years ago by Russell Taylor

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Russell Taylor

Use compilers found by FindMPI instead of hardcoding. Re #4961.

Allows overriding via the cmake variable, if desired.

Changeset: 32765c3e4289da260e84d7a9884e4b32e65dd4c3

comment:3 Changed 9 years ago by Russell Taylor

  • Status changed from accepted to verify
  • Resolution set to fixed

comment:4 Changed 9 years ago by Russell Taylor

Use compilers found by FindMPI instead of hardcoding. Re #4961.

Allows overriding via the cmake variable, if desired.

Changeset: 32765c3e4289da260e84d7a9884e4b32e65dd4c3

comment:5 Changed 9 years ago by Russell Taylor

Re #4961. No need to link directly to MPI libraries.

Just have to link to boost-mpi.

Changeset: fb80633433588277448ef9ee1f72f8b8dbceed56

comment:6 Changed 9 years ago by Russell Taylor

Re #4961. No need to link directly to MPI libraries.

Just have to link to boost-mpi.

Changeset: fb80633433588277448ef9ee1f72f8b8dbceed56

comment:7 Changed 8 years ago by Stuart Campbell

  • Status changed from verify to verifying
  • Tester set to Stuart Campbell

comment:8 Changed 8 years ago by Stuart Campbell

  • Status changed from verifying to closed

Looks ok to me.

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 5807

Note: See TracTickets for help on using tickets.