Skip to content

{bio}[goolf/1.7.20] fastQValidator v0.1.1a-20151214#3192

Merged
boegel merged 1 commit intoeasybuilders:developfrom
pescobar:20160608131717_new_pr_fastQValidator011a20151214
Aug 5, 2016
Merged

{bio}[goolf/1.7.20] fastQValidator v0.1.1a-20151214#3192
boegel merged 1 commit intoeasybuilders:developfrom
pescobar:20160608131717_new_pr_fastQValidator011a20151214

Conversation

@pescobar
Copy link
Copy Markdown
Member

@pescobar pescobar commented Jun 8, 2016

(created using eb --new-pr)

@pescobar
Copy link
Copy Markdown
Member Author

pescobar commented Jun 8, 2016

Test report by @pescobar
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
login18.cluster.bc2.ch - Linux centos 6.5, Intel(R) Xeon(R) CPU E5620 @ 2.40GHz, Python 2.6.6
See https://gist.github.com/8933b8a8cd9b8cbb4ddd8b28a99e34cb for a full test report.


sources = [
'%s.tar.gz' % fastqvalidator_git_commit,
'%s.tar.gz' % libstatgen_git_commit
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

because of the way EasyBuild works, there is a non-zero probability
that you may end up hitting a tarball with the same commit id from the other source_url;
for this reason, I'd consider it recommended to provide the tarball hashes, as extra safety... <0.02 cents...

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@pescobar good to go once checksums have been added for the sources

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

actually, thinking about this again, it would be a pretty weird coincidence if you could download the same commit from another repo...

so, I'll go ahead and merge this as-is, thanks @pescobar!

@boegel boegel added this to the v2.9.0 milestone Jul 8, 2016
@boegel
Copy link
Copy Markdown
Member

boegel commented Jul 13, 2016

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2689.swalot.os - Linux centos linux 7.2.1511, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/d7b9bf9854b45417474c1bc16fe22925 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jul 13, 2016

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Aug 5, 2016

lgtm

@boegel
Copy link
Copy Markdown
Member

boegel commented Aug 5, 2016

Going in, thanks @pescobar!

@boegel boegel merged commit e993491 into easybuilders:develop Aug 5, 2016
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