Skip to content

reinstate Velvet easyconfigs that were removed in #2495#2729

Merged
boegel merged 1 commit intoeasybuilders:developfrom
boegel:reinstate_velvet_easyconfigs
Mar 16, 2016
Merged

reinstate Velvet easyconfigs that were removed in #2495#2729
boegel merged 1 commit intoeasybuilders:developfrom
boegel:reinstate_velvet_easyconfigs

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Mar 16, 2016

In #2495, existing easyconfigs were renamed because Perl/BioPerl was added as a dependency.

But since our policy is not to remove existing easyconfigs from the repository (unless there's a good reason for it), the existing Velvet easyconfigs shouldn't have been removed/renamed, but new ones should've been added.

This PR fixes that by readding the original Velvet easyconfigs.

@boegel boegel added this to the v2.7.0 milestone Mar 16, 2016
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 16, 2016

Jenkins: test this please

@hpcugentbot
Copy link
Copy Markdown

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7159/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.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 16, 2016

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

boegel added a commit that referenced this pull request Mar 16, 2016
reinstate Velvet easyconfigs that were removed in #2495
@boegel boegel merged commit e58426a into easybuilders:develop Mar 16, 2016
@boegel boegel deleted the reinstate_velvet_easyconfigs branch March 16, 2016 19:25
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