Skip to content

use $CC,$CXX rather than $I_MPI_CC,$I_MPI_CXX in patch for OpenFOAM 4.0#3703

Merged
boegel merged 1 commit intoeasybuilders:developfrom
boegel:OF4_fix_patch
Oct 25, 2016
Merged

use $CC,$CXX rather than $I_MPI_CC,$I_MPI_CXX in patch for OpenFOAM 4.0#3703
boegel merged 1 commit intoeasybuilders:developfrom
boegel:OF4_fix_patch

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Oct 25, 2016

@wpoely86 just to ensure that this patch can be reused with other (non-Intel MPI) toolchains

@boegel boegel added this to the 3.0 milestone Oct 25, 2016
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Oct 25, 2016

good to go, thanks for the review @wpoely86!

$ eb OpenFOAM-4.0-intel-2016b.eb  -x | egrep 'export CC=|export CXX=|export I_MPI_CC=|export I_MPI_CXX='
  export CC="icc"
  export CXX="icpc"
  export I_MPI_CC="icc"
  export I_MPI_CXX="icpc"

@boegel boegel merged commit 7a9715e into easybuilders:develop Oct 25, 2016
@boegel boegel deleted the OF4_fix_patch branch October 25, 2016 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wpoely86 wpoely86 wpoely86 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

3.0

Development

Successfully merging this pull request may close these issues.

2 participants

@boegel @wpoely86