Skip to content

Changing dry-run output to include paths for requirements in make_module_req#1520

Merged
boegel merged 11 commits intoeasybuilders:developfrom
rjeschmi:dryrun-req-search
Jan 22, 2016
Merged

Changing dry-run output to include paths for requirements in make_module_req#1520
boegel merged 11 commits intoeasybuilders:developfrom
rjeschmi:dryrun-req-search

Conversation

@rjeschmi
Copy link
Copy Markdown
Contributor

@hpcugentbot
Copy link
Copy Markdown

Automatic reply from Jenkins: Can I test this?

@boegel boegel added this to the v2.6.0 milestone Dec 16, 2015
Comment thread easybuild/framework/easyblock.py Outdated
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.

style: spaces around =

@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 16, 2015

Jenkins: ok to test

@hpcugentbot
Copy link
Copy Markdown

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

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

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

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

Comment thread easybuild/framework/easyblock.py Outdated
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.

line too long

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 17, 2016

@rjeschmi: some minor open remarks, let's get this merged?

@hpcugentbot
Copy link
Copy Markdown

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

boegel commented Jan 21, 2016

@rjeschmi: some enhancements in rjeschmi#13, please review/merge, let's get this in :)

prettify output of make_module_req in dry run mode
@rjeschmi
Copy link
Copy Markdown
Contributor Author

there is still something off, give me a second. I'm getting output that shouldn't be run in acutal builds.

@hpcugentbot
Copy link
Copy Markdown

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

boegel commented Jan 22, 2016

Going in, thanks @rjeschmi!

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