Skip to content

sync Bowtie2 easyconfigs, consistently use Bowtie2 easyblock#4380

Merged
boegel merged 1 commit intoeasybuilders:developfrom
boegel:Bowtie2_sync
Mar 23, 2017
Merged

sync Bowtie2 easyconfigs, consistently use Bowtie2 easyblock#4380
boegel merged 1 commit intoeasybuilders:developfrom
boegel:Bowtie2_sync

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Mar 21, 2017

@boegel boegel added this to the 3.2.0 milestone Mar 21, 2017
@boegel boegel mentioned this pull request Mar 21, 2017
Merged
rjeschmi
rjeschmi approved these changes Mar 21, 2017
View reviewed changes
Copy link
Copy Markdown
Contributor

@rjeschmi rjeschmi left a comment

Choose a reason for hiding this comment

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

Looks good to me providing all these old versions build :)

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 21, 2017

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

@verdurin
Copy link
Copy Markdown
Member

verdurin commented Mar 21, 2017

@boegel could we have a foss-2016b version too? This makes me realise I never submitted my local one...

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 21, 2017

@verdurin Sure, but let's do that in a separate PR...

@verdurin
Copy link
Copy Markdown
Member

verdurin commented Mar 21, 2017

@boegel - okay, otherwise looks fine to me, and a nice improvement.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 23, 2017

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

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 23, 2017

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

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 23, 2017

tested on top of latest revision of easybuilders/easybuild-easyblocks#1146, so good to go, thanks for the feedback everyone!

@boegel boegel merged commit d6a8d9a into easybuilders:develop Mar 23, 2017
@boegel boegel deleted the Bowtie2_sync branch March 23, 2017 20:36
@boegel boegel mentioned this pull request Aug 10, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@rjeschmi rjeschmi rjeschmi approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

3.2.0

Development

Successfully merging this pull request may close these issues.

3 participants

@boegel @verdurin @rjeschmi