Skip to content

enhance impi easyblock to handle install subdir cr*p for impi v5.0.1 too#465

Merged
boegel merged 3 commits intoeasybuilders:developfrom
boegel:impi5
Sep 2, 2014
Merged

enhance impi easyblock to handle install subdir cr*p for impi v5.0.1 too#465
boegel merged 3 commits intoeasybuilders:developfrom
boegel:impi5

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Sep 2, 2014

No description provided.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 2, 2014

@wpoely86: required for easybuilders/easybuild-easyconfigs#1051, please review?

Tested with both impi 4.1.1 and 5.0.2, other versions should be unaffected.

@wpoely86
Copy link
Copy Markdown
Member

wpoely86 commented Sep 2, 2014

Looks good

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 2, 2014

Going in, thanks @wpoely86!

boegel added a commit that referenced this pull request Sep 2, 2014
enhance impi easyblock to handle install subdir cr*p for impi v5.0.1 too
@boegel boegel merged commit e8d2813 into easybuilders:develop Sep 2, 2014
@boegel boegel deleted the impi5 branch September 2, 2014 14:01
@wpoely86
Copy link
Copy Markdown
Member

wpoely86 commented Sep 3, 2014

From the release notes of IMPI 5.0.1:
Directory structure update. New links have been added to always point to the most recently installed version of the Intel(R) MPI Library.

So probably a keeper. We should make the version check to work for all >= 5.0.1?

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 3, 2014

@wpoely86: see #469

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