Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add DISCOVAR/DISCOVARdenovo-52488-foss-2015a.eb #1932

Merged

Conversation

molden
Copy link
Contributor

@molden molden commented Aug 31, 2015

add DISCOVAR/DISCOVARdenovo-52488-foss-2015a.eb and dependency j/jemalloc/jemalloc-3.6.0-foss-2015a.eb.

@hpcugentbot
Copy link
Contributor

Automatic reply from Jenkins: Can I test this?

@@ -0,0 +1,22 @@
name = 'DISCOVARdenovo'
Copy link
Member

Choose a reason for hiding this comment

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

please make sure this makes the name of the directory the easyconfig file is in

@boegel
Copy link
Member

boegel commented Sep 2, 2015

Jenkins: ok to test

@hpcugentbot
Copy link
Contributor

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

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4255/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

boegel commented Sep 3, 2015

@molden: same here: missing easyblock spec in the DISCOVARdenovo easyconfig, or does it require a custom easyblock?

@hpcugentbot
Copy link
Contributor

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

boegel commented Sep 7, 2015

@molden: please remove easybuild/easyconfigs/d/DISCOVAR/DISCOVARdenovo-52488-foss-2015a.eb, since easybuild/easyconfigs/d/DISCOVARdenovo/DISCOVARdenovo-52488-foss-2015a.eb replaces it...

git rm easybuild/easyconfigs/d/DISCOVARdenovo/DISCOVARdenovo-52488-foss-2015a.eb

@boegel
Copy link
Member

boegel commented Sep 7, 2015

Sorry, I meant

git rm easybuild/easyconfigs/d/DISCOVAR/DISCOVARdenovo-52488-foss-2015a.eb

@hpcugentbot
Copy link
Contributor

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

boegel commented Sep 8, 2015

Test report by @boegel
FAILED
Build succeeded for 1 out of 2 (2 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/fd856f33cab3f7b60bd4 for a full test report.

edit: failed because auto-download didn't work since downloading from ftp:// is blocked on this system

@boegel
Copy link
Member

boegel commented Sep 8, 2015

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 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/0ed4f70ca59de4955e24 for a full test report.

@boegel
Copy link
Member

boegel commented Sep 8, 2015

Going in, thanks @molden!

boegel added a commit that referenced this pull request Sep 8, 2015
…foss-2015a

add DISCOVAR/DISCOVARdenovo-52488-foss-2015a.eb
@boegel boegel merged commit b2c6512 into easybuilders:develop Sep 8, 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