{bio} [foss-2016a] annovar-2016Feb01-foss-2016a-Perl-5.22.1 (REVIEW)#3010
Conversation
|
Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7909/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. |
|
|
||
| # Registration required for download | ||
| # http://www.openbioinformatics.org/annovar/annovar_download_form.php | ||
| sources = ['%(name)s.latest.tar.gz'] |
There was a problem hiding this comment.
please enforce renaming of the tarball using %(name)s-%(version)s.tar.gz, and include a checksum (I have 2b26e514c54a847bfdcf4ce57e43f95c as MD5 for the version download today)
|
Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7954/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. |
| easyblock = 'Tarball' | ||
|
|
||
| name = 'annovar' | ||
| version = '2016Feb01' |
There was a problem hiding this comment.
mention how you came to this ($DATE in table_annovar.pl?)
also, I'm more in favour of 20160201, since that'll sort properly
There was a problem hiding this comment.
I would prefer that sort of version too, but I took the current version from the website - that's their naming scheme.
There was a problem hiding this comment.
ok, then please include a comment above version explaining where you got it from
|
Test report by @boegel |
|
Going in, thanks @verdurin! |
(created using
eb --new-pr)annovar 2016Feb01