Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

skip unneeded unuse/use commands on tail of $MODULEPATH in check_module_path #1813

Merged
merged 2 commits into from Jun 29, 2016
Merged

skip unneeded unuse/use commands on tail of $MODULEPATH in check_module_path #1813

merged 2 commits into from Jun 29, 2016

Conversation

boegel
Copy link
Member

@boegel boegel commented Jun 20, 2016

fix for the issue reported in #1804, we should avoid unneeded unuse commands, since they can have unwanted side-effects

@boegel boegel added this to the v2.9.0 milestone Jun 20, 2016
@boegel boegel mentioned this pull request Jun 20, 2016
Closed
@boegel
Copy link
Member Author

boegel commented Jun 29, 2016

@wpoely86 please review?

@wpoely86
Copy link
Member

👍

@boegel
Copy link
Member Author

boegel commented Jun 29, 2016

Thanks for the review @wpoely86!

@boegel boegel merged commit b5c1dbb into easybuilders:develop Jun 29, 2016
@boegel boegel deleted the skip_unuse_use_check_module_path branch June 29, 2016 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v2.9.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@boegel @wpoely86