Skip to content

renamed blast easyconfigs (REVIEW)#1443

Merged
boegel merged 6 commits intoeasybuilders:developfrom
pescobar:blast-name-fix
Mar 6, 2015
Merged

renamed blast easyconfigs (REVIEW)#1443
boegel merged 6 commits intoeasybuilders:developfrom
pescobar:blast-name-fix

Conversation

@pescobar
Copy link
Copy Markdown
Member

@pescobar pescobar commented Mar 4, 2015

No description provided.

@pescobar
Copy link
Copy Markdown
Member Author

pescobar commented Mar 4, 2015

I reported this issue here #1442

I also added a new easyconfig for latest BLAST+

@hpcugentbot
Copy link
Copy Markdown

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 4, 2015

@pescobar: two issues to fix:

EasyBuildError: 'EasyBuild crashed with an error (at easybuild/tools/robot.py:232 in
resolve_dependencies): Irresolvable dependencies encountered:
BLAST/2.2.28-goolf-1.4.10-Python-2.7.3, BLAST/2.2.28-ictce-5.3.0-Python-2.7.3'
AssertionError: Patch file /data/jenkins/workspace/easybuild-easyconfigs-pr-builder/
easybuild/easyconfigs/b/BLAST/BLAST+-2.2.30_ictce-fixes.patch is
available for BLAST+-2.2.30-goolf-1.4.10.eb

@hpcugentbot
Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/2474/
Test PASSed.

@boegel boegel changed the title renamed blast easyconfigs {REVIEW} renamed blast easyconfigs (REVIEW) Mar 5, 2015
@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 5, 2015

Test report by @boegel
FAILED
Build succeeded for 0 out of 14 (13 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/efe4043216af0847dcd8 for a full test report.

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.

please drop this

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 5, 2015

@pescobar: please move the BLAST+ easyconfigs into a directory named BLAST+?

@hpcugentbot
Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/2484/
Test PASSed.

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 5, 2015

Test report by @boegel
FAILED
Build succeeded for 11 out of 14 (13 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/d2c6bc5c846990c72ea1 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 5, 2015

trouble in paradise... will check later what went wrong

@pescobar: can you spot the problem, and maybe fix it?

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 5, 2015

@pescobar: BLAST+-2.2.30-goolf-1.4.10.eb doesn't work on my end...

/bin/sh: line 2: /usr/bin/basename: No such file or directory
/bin/ln: accessing `atic.a': No such file or directory

You're also triggering a rebuild of HH-suite-2.0.16-goolf-1.4.10.eb, which is known to be broken (see #414), but that's not the fault of this PR.

So, my suggestion would be: take out the new easyconfig for BLAST+ v2.2.30, and contribute that via a separate PR, so we can get the naming fix included for BLAST+.

@hpcugentbot
Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/2493/
Test PASSed.

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 5, 2015

Test report by @boegel
FAILED
Build succeeded for 11 out of 13 (12 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/f2e33e9d8687c844ce2c for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 6, 2015

All BLAST easyconfigs (still) work, the HH-suite failure is a known issue, so good to go.

Thanks @pescobar!

boegel added a commit that referenced this pull request Mar 6, 2015
renamed blast easyconfigs (REVIEW)
@boegel boegel merged commit d3ea84d into easybuilders:develop Mar 6, 2015
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