Skip to content

Conversation

@boegel
Copy link
Member

@boegel boegel commented Jul 2, 2015

@boegel
Copy link
Member Author

boegel commented Jul 2, 2015

@wpoely86 or @JensTimmerman: please review?

@hpcugentbot
Copy link

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

See https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/1039/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
Member Author

boegel commented Jul 2, 2015

(note: unit tests fail because easybuilders/easybuild-framework#1310 is missing, but please do review)

@wpoely86
Copy link
Member

wpoely86 commented Jul 2, 2015

looks good

@boegel
Copy link
Member Author

boegel commented Jul 2, 2015

Jenkins: test this please

@hpcugentbot
Copy link

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

See https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/1040/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
Member Author

boegel commented Jul 2, 2015

Jenkins: test this please

@hpcugentbot
Copy link

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

boegel commented Jul 2, 2015

Thanks for the review @wpoely86!

boegel added a commit that referenced this pull request Jul 2, 2015
fix setting of $INTEL_LICENSE_FILE for port@server values
@boegel boegel merged commit 0485764 into easybuilders:develop Jul 2, 2015
@boegel boegel deleted the intel_license_server_env_var branch July 2, 2015 11:30
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