Skip to content

RMBlast#3142

Merged
boegel merged 15 commits intoeasybuilders:developfrom
rug-cit-hpc:RMBlast-2.2.28
Jun 6, 2016
Merged

RMBlast#3142
boegel merged 15 commits intoeasybuilders:developfrom
rug-cit-hpc:RMBlast-2.2.28

Conversation

@RvDijk
Copy link
Copy Markdown
Contributor

@RvDijk RvDijk commented May 29, 2016

Dependency for RepeatMasker

easyblock = 'ConfigureMake'

name = 'RMBlast'
version = '2.2.28-linux64'
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 would make this 2.2.28, and move the -linux64 part to the versionsuffix (or maybe just remove it?)

@boegel boegel added this to the v2.x milestone May 29, 2016
checksums = [('md5', 'fb5f4e2e02ffcb1b17af2e9f206c5c22')]

dependencies = [
('BLAST+', '2.3.0', versionsuffix, ('foss', '2016a')),
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.

@RvDijk Specify the toolchain via toolchain = instead?

@boegel
Copy link
Copy Markdown
Member

boegel commented May 30, 2016

Test report by @boegel
FAILED
Build succeeded for 2 out of 3 (3 easyconfigs in this PR)
node2542.golett.os - Linux centos linux 7.2.1511, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/5e8459d7e7c7b07c667112f066f636aa for a full test report.

@boegel boegel modified the milestones: v2.9.0, v2.x Jun 3, 2016
@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 3, 2016

Test report by @boegel
FAILED
Build succeeded for 2 out of 3 (3 easyconfigs in this PR)
node2101.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/9ba3400103f9e422b4bd7e8ad49aa44f for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 3, 2016

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

sources = ['ncbi-rmblastn-%(version)s-src.tar.gz']
checksums = [('md5', 'fb5f4e2e02ffcb1b17af2e9f206c5c22')]

# ('BLAST+', '2.3.0', versionsuffix),
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.

@RvDijk so, this is not needed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@boegel I figured that RMBlast has all functionalities of BLAST+ including some others, thus does not depend on BLAST+ since it is already in it.

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.

@RvDijk Yet, a quick look at http://www.repeatmasker.org/RMBlast.html tells me that BLAST+ is actually required to install RMBlast?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@boegel I did think the same, however in the release notes they say that:
"RMBlast 2.2.27+ First integrated release of NCBI BLAST+ toolkit and RMBlast."
And they also argue that both installations are needed for precompiled (not from source) installations:
"If you choose to use the pre-compiled binaries you will need both for use with RepeatMasker/RepeatModeler."
If you think that I misunderstood these lines, then I will include the BLAST+ dependency.

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.

@RvDijk Well, if it builds/installs and works, it should be fine without providing BLAST+...

In that case, please remove this commented out line.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 6, 2016

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2096.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/3427de88d52b5c823791acd087b291ff for a full test report.

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.

2 participants