Skip to content

fix combined use of --hide-deps and hiddendependencies#1280

Merged
boegel merged 2 commits intoeasybuilders:developfrom
boegel:fix_hide_deps
May 11, 2015
Merged

fix combined use of --hide-deps and hiddendependencies#1280
boegel merged 2 commits intoeasybuilders:developfrom
boegel:fix_hide_deps

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented May 11, 2015

fix for bug #1279, reported by @pescobar

@hpcugentbot
Copy link
Copy Markdown

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

@wpoely86
Copy link
Copy Markdown
Member

looks fine I think

@boegel
Copy link
Copy Markdown
Member Author

boegel commented May 11, 2015

Thanks for the review @wpoely86. The unit test confirms it's the proper fix, before this patch you'd run into an error message as mentioned in #1279

boegel added a commit that referenced this pull request May 11, 2015
fix combined use of --hide-deps and hiddendependencies
@boegel boegel merged commit 003c275 into easybuilders:develop May 11, 2015
@boegel boegel deleted the fix_hide_deps branch May 11, 2015 20:15
boegel added a commit to boegel/easybuild-framework that referenced this pull request May 13, 2015
fix combined use of --hide-deps and hiddendependencies
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