Skip to content

{bio}[goolf/1.4.10] blasr smrtanalysis-2.1 (REVIEW)#1662

Merged
boegel merged 5 commits intoeasybuilders:developfrom
boegel:pr699
Jun 3, 2015
Merged

{bio}[goolf/1.4.10] blasr smrtanalysis-2.1 (REVIEW)#1662
boegel merged 5 commits intoeasybuilders:developfrom
boegel:pr699

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Jun 3, 2015

wrapper PR for #699 by @pforai

ocaisa
ocaisa reviewed Jun 3, 2015
View reviewed changes
Comment thread easybuild/easyconfigs/b/blasr/blasr-smrtanalysis-2.1-goolf-1.4.10.eb Outdated
Copy link
Copy Markdown
Member

@ocaisa ocaisa Jun 3, 2015

Choose a reason for hiding this comment

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

This line is too long...120 char limit

Copy link
Copy Markdown
Member Author

@boegel boegel Jun 3, 2015

Choose a reason for hiding this comment

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

sorry, wasn't aware of that ;-)

Fixed.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Jun 3, 2015

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

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Jun 3, 2015

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

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Jun 3, 2015

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

@rjeschmi
Copy link
Copy Markdown
Contributor

rjeschmi commented Jun 3, 2015

Test report by @rjeschmi
SUCCESS
Build succeeded for 3 out of 3 (1 easyconfigs in this PR)
Linux centos 6.6, AMD Opteron(tm) Processor 6380, Python 2.6.6
See https://gist.github.com/33b72c587bdd4e9d4fe8 for a full test report.

@rjeschmi
Copy link
Copy Markdown
Contributor

rjeschmi commented Jun 3, 2015

@boegel Seems fine (failures were my fault).

fgeorgatos
fgeorgatos reviewed Jun 3, 2015
View reviewed changes
Comment thread easybuild/easyconfigs/b/blasr/blasr-smrtanalysis-2.1-goolf-1.4.10.eb Outdated
Copy link
Copy Markdown
Contributor

@fgeorgatos fgeorgatos Jun 3, 2015

Choose a reason for hiding this comment

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

remove extra line

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Jun 3, 2015

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

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

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Jun 3, 2015

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

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

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Jun 3, 2015

Thanks for the review @ocaisa and @fgeorgatos, and for testing @rjeschmi!
And @pforai for kicking this off in #699 ;-)

Going in!

boegel added a commit that referenced this pull request Jun 3, 2015
@boegel
Merge pull request #1662 from boegel/pr699
dc299a5 
{bio}[goolf/1.4.10] blasr smrtanalysis-2.1 (REVIEW)
@boegel boegel merged commit dc299a5 into easybuilders:develop Jun 3, 2015
@boegel boegel deleted the pr699 branch June 3, 2015 15:02
@boegel boegel mentioned this pull request Jun 3, 2015
Merged
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.

5 participants

@boegel @hpcugentbot @rjeschmi @fgeorgatos @ocaisa