Skip to content

fix DB dep in DB_File easyconfigs#2539

Merged
boegel merged 6 commits intoeasybuilders:developfrom
boegel:DB_File_fix
Feb 22, 2016
Merged

fix DB dep in DB_File easyconfigs#2539
boegel merged 6 commits intoeasybuilders:developfrom
boegel:DB_File_fix

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Feb 22, 2016

No description provided.

@boegel boegel added this to the v2.7.0 milestone Feb 22, 2016
@hpcugentbot
Copy link
Copy Markdown

Easyconfigs unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/6476/console for more details.

Please fix the reported issues by pushing additional commits to the branch corresponding with this pull request; contact @boegel if you're not sure what to do.

@hpcugentbot
Copy link
Copy Markdown

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

@hpcugentbot
Copy link
Copy Markdown

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

@hpcugentbot
Copy link
Copy Markdown

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

Test report by @boegel
SUCCESS
Build succeeded for 6 out of 6 (6 easyconfigs in this PR)
Linux SL 6.7, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/fd149816a81b43629378 for a full test report.

@hpcugentbot
Copy link
Copy Markdown

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

@hpcugentbot
Copy link
Copy Markdown

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

source_urls = ['http://www.cpan.org/modules/by-module/DB_File/PMQS']
sources = [SOURCE_TAR_GZ]

dependencies = [('DB', '2.7.7')]
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.

this is a version change?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

well, not really...

this dep was not used, since dependencies is redefined below

and when I included the 2.7.7 version of DB properly, the tests for DB_File were failing

with 4.8.30, they pass...

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Feb 22, 2016

Thanks for the review @wpoely86!

boegel added a commit that referenced this pull request Feb 22, 2016
fix DB dep in DB_File easyconfigs
@boegel boegel merged commit 3761a2b into easybuilders:develop Feb 22, 2016
@boegel boegel deleted the DB_File_fix branch February 22, 2016 15:57
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