Skip to content

{cae}[dummy] Samcef 17.0-03 (REVIEW)#1941

Merged
boegel merged 1 commit intoeasybuilders:developfrom
boegel:Samcef
Sep 2, 2015
Merged

{cae}[dummy] Samcef 17.0-03 (REVIEW)#1941
boegel merged 1 commit intoeasybuilders:developfrom
boegel:Samcef

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Sep 2, 2015

@wpoely86
Copy link
Copy Markdown
Member

wpoely86 commented Sep 2, 2015

looks fine

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Sep 2, 2015

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4250/
Easyconfigs unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4250/console for more details.

Please fix the reported issues by pushing additional commits to the branch corresponding with this pull request; contact @boegel if you're not sure what to do.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 2, 2015

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
Linux SL 6.7, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/b2395ca2068d8985f2e9 for a full test report.

@boegel boegel mentioned this pull request Sep 2, 2015
Merged
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 2, 2015

Jenkins: test this please

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Sep 2, 2015

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4251/
Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4251/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.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 2, 2015

Thanks for the review @wpoely86!

boegel added a commit that referenced this pull request Sep 2, 2015
@boegel
Merge pull request #1941 from boegel/Samcef
a17a20c 
{cae}[dummy] Samcef 17.0-03 (REVIEW)
@boegel boegel merged commit a17a20c into easybuilders:develop Sep 2, 2015
@boegel boegel deleted the Samcef branch September 2, 2015 15:17
fgeorgatos
fgeorgatos reviewed Sep 2, 2015
View reviewed changes
Comment thread easybuild/easyconfigs/s/Samcef/Samcef-17.0-03.eb
Copy link
Copy Markdown
Contributor

@fgeorgatos fgeorgatos Sep 2, 2015

Choose a reason for hiding this comment

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

oops, hardwired version, better to avoid that!

Copy link
Copy Markdown
Member Author

@boegel boegel Sep 2, 2015

Choose a reason for hiding this comment

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

aah, good point, I'll get that fixed, thanks @fgeorgatos!

@wpoely86: -1 for you ;)

Copy link
Copy Markdown
Member Author

@boegel boegel Sep 2, 2015

Choose a reason for hiding this comment

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

see #1942

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@boegel @wpoely86 @hpcugentbot @fgeorgatos