Skip to content

Add .eb file for verifyBamID-1.1.2 with goolf 1.7.20#1675

Merged
boegel merged 7 commits intoeasybuilders:developfrom
npklein:verifyBamID-1.1.2-goolf-1.7.20
Jun 17, 2015
Merged

Add .eb file for verifyBamID-1.1.2 with goolf 1.7.20#1675
boegel merged 7 commits intoeasybuilders:developfrom
npklein:verifyBamID-1.1.2-goolf-1.7.20

Conversation

@npklein
Copy link
Copy Markdown
Contributor

@npklein npklein commented Jun 5, 2015

I did something wrong when making branches, and got GenotypeHarmonizer in this branch. I hope this is not a problem, otherwise please let me know.

Niek

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Jun 5, 2015

Automatic reply from Jenkins: Can I test this?

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 5, 2015

Jenkins: ok to test

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 5, 2015

@npklein: as long as we get #1672 merged first, it's fine as is

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Jun 5, 2015

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/3431/
Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/3431/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
boegel reviewed Jun 5, 2015
View reviewed changes
Comment thread easybuild/easyconfigs/v/verifyBamID/verifyBamID-1.1.2-goolf-1.7.20.eb Outdated
Copy link
Copy Markdown
Member

@boegel boegel Jun 5, 2015

Choose a reason for hiding this comment

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

remove double newline

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Jun 5, 2015

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

boegel commented Jun 5, 2015

@npklein: can you sync this branch with current develop, now that #1672 is merged in, so we have a clear view what is being changed here?

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Jun 5, 2015

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/3441/
Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/3441/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
boegel reviewed Jun 5, 2015
View reviewed changes
Comment thread easybuild/easyconfigs/g/GenotypeHarmonizer/GenotypeHarmonizer-1.4.14-Java-1.7.0_80.eb Outdated
Copy link
Copy Markdown
Member

@boegel boegel Jun 5, 2015

Choose a reason for hiding this comment

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

this should not be here, toolchain is already defined higher up in this easyconfig

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Jun 8, 2015

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

boegel commented Jun 17, 2015

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 17, 2015

Going in, thanks @npklein!

boegel added a commit that referenced this pull request Jun 17, 2015
@boegel
Merge pull request #1675 from npklein/verifyBamID-1.1.2-goolf-1.7.20
2c86437 
Add .eb file for verifyBamID-1.1.2 with goolf 1.7.20
@boegel boegel merged commit 2c86437 into easybuilders:develop Jun 17, 2015
@npklein npklein deleted the verifyBamID-1.1.2-goolf-1.7.20 branch June 17, 2015 08:25
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.

3 participants

@npklein @hpcugentbot @boegel