Skip to content

impi mic mpivars fix#662

Merged
boegel merged 1 commit intoeasybuilders:developfrom
stachon:master
Aug 13, 2015
Merged

impi mic mpivars fix#662
boegel merged 1 commit intoeasybuilders:developfrom
stachon:master

Conversation

@stachon
Copy link
Copy Markdown
Contributor

@stachon stachon commented Aug 13, 2015

the bash version was incorrect

resubmitting #655, this time to develop branch

the bash version was incorrect
@hpcugentbot
Copy link
Copy Markdown

Automatic reply from Jenkins: Can I test this?

@stachon stachon mentioned this pull request Aug 13, 2015
@stachon
Copy link
Copy Markdown
Contributor Author

stachon commented Aug 13, 2015

Jenkins: ok to test

@boegel
Copy link
Copy Markdown
Member

boegel commented Aug 13, 2015

@stachon: our bot only listens to people it knows :)

@boegel
Copy link
Copy Markdown
Member

boegel commented Aug 13, 2015

Jenkins: ok to test

@hpcugentbot
Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/1102/
Easyblocks unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/1102/console for more details).

This pull request is now ready for review/testing.

Please try and find someone who can tackle this; contact @boegel if you're not sure what to do.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Urgh, this should never have made it in like this, thanks @stachon!

How did we miss this in #629 @ocaisa?

@boegel
Copy link
Copy Markdown
Member

boegel commented Aug 13, 2015

Going in, thanks @stachon, great catch.

boegel added a commit that referenced this pull request Aug 13, 2015
@boegel boegel merged commit 36234b9 into easybuilders:develop Aug 13, 2015
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.

3 participants