Skip to content

{bio}[intel/2018a] GenomeTester4 v4.0#6970

Merged
boegel merged 4 commits intoeasybuilders:developfrom
verdurin:20181006103918_new_pr_GenomeTester404
Oct 11, 2018
Merged

{bio}[intel/2018a] GenomeTester4 v4.0#6970
boegel merged 4 commits intoeasybuilders:developfrom
verdurin:20181006103918_new_pr_GenomeTester404

Conversation

@verdurin
Copy link
Copy Markdown
Member

@verdurin verdurin commented Oct 6, 2018

(created using eb --new-pr)

@verdurin verdurin added the new label Oct 6, 2018
files_to_copy = [
(['bin/glistcompare', 'bin/glistmaker', 'bin/glistquery',
'bin/gmer_caller', 'bin/gmer_counter'], "bin"),
]
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.

@verdurin You can just make this:

files_to_copy = ['bin']

To make it copy the whole bin directory, and then check for specific binaries in the sanity check (which you're already doing); that will avoid having to update this in case additional binaries are added in future versions...

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 6, 2018

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
gligar02.gligar.os - Linux centos linux 7.5.1804, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/dd160e2fbf26ef276293a5e1164e65f5 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 7, 2018

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2410.golett.os - Linux centos linux 7.5.1804, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/b721584f3491135358f74b33951b4a4e for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 7, 2018

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
gligar01.gligar.os - Linux centos linux 7.5.1804, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/937ff421496c3f7b11570de10caf5b8f for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 7, 2018

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node3121.skitty.os - Linux centos linux 7.5.1804, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 2.7.5
See https://gist.github.com/9363bd64a6bd9df85c464ed07a035a66 for a full test report.

toolchain = {'name': 'intel', 'version': '2018a'}

source_urls = ['https://github.com/bioinfo-ut/GenomeTester4/archive/']
sources = ['Version_4_0.tar.gz']
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.

@verdurin Hmm, I overlooked this earlier... How does this relate to the 0.4 version exactly? Should we use %(version_minor)s here?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@boegel yes, it appears I made that up in attempt to reconstruct a release history that doesn't really exist.

If you look at https://github.com/bioinfo-ut/GenomeTester4/releases it's described as test.

Calling it version 4 feels wrong, nevertheless.

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.

I'd say 4.0 is the right version, and then you can use:

sources = ['Version_%(version_major)s_%(version_minor)s.tar.gz']

@verdurin verdurin changed the title {bio}[intel/2018a] GenomeTester4 v0.4 {bio}[intel/2018a] GenomeTester4 v4.0 Oct 10, 2018
@boegel boegel added this to the 3.7.1 milestone Oct 11, 2018
@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 11, 2018

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 11, 2018

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node3100.skitty.os - Linux centos linux 7.5.1804, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 2.7.5
See https://gist.github.com/c2d7bd864ce1c341d3ee6515c84d7448 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 11, 2018

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
gligar03.gligar.os - Linux centos linux 7.5.1804, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/e099fa95065471259e022c43eb33e6a7 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 11, 2018

Going in, thanks @verdurin!

@boegel boegel merged commit cda7baf into easybuilders:develop Oct 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants