Skip to content

Cope with wrong modulepath#969

Merged
boegel merged 3 commits intoeasybuilders:developfrom
boegel:cope_with_wrong_modulepath
Jul 7, 2014
Merged

Cope with wrong modulepath#969
boegel merged 3 commits intoeasybuilders:developfrom
boegel:cope_with_wrong_modulepath

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Jul 7, 2014

fixes issue reported by @besserox on the EasyBuild mailing list:


$ eb --debug -S GCC
== temporary log file in case of crash
/tmp/easybuild-oQ2Uot/easybuild-UpUFfC.log
ERROR: EasyBuild crashed with an error (at
easybuild/easybuild-develop/easybuild-framework/easybuild/tools/filetools.py:800
in mkdir): Failed to create directory
/usr/Modules/$MODULE_VERSION/modulefiles: [Errno 13] Permission
denied: '/usr/Modules'

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Jul 7, 2014

@besserox/@wpoely86: please review?
@pescobar: I think you ran into this problem too?

wpoely86
wpoely86 reviewed Jul 7, 2014
View reviewed changes
Comment thread easybuild/tools/modules.py Outdated
Copy link
Copy Markdown
Member

@wpoely86 wpoely86 Jul 7, 2014

Choose a reason for hiding this comment

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

will os.path.exists(p) not also give false if len(p) == 0?

Copy link
Copy Markdown
Member Author

@boegel boegel Jul 7, 2014

Choose a reason for hiding this comment

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

yes, excellent remark, will fix

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Jul 7, 2014

Thanks for the review @wpoely86!

boegel added a commit that referenced this pull request Jul 7, 2014
@boegel
Merge pull request #969 from boegel/cope_with_wrong_modulepath
3cd7c9d 
Cope with wrong modulepath
@boegel boegel merged commit 3cd7c9d into easybuilders:develop Jul 7, 2014
@boegel boegel deleted the cope_with_wrong_modulepath branch July 7, 2014 20:33
@besserox
Copy link
Copy Markdown
Contributor

besserox commented Jul 7, 2014

This change fixes the issue for me. Thanks!

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

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@boegel @besserox @wpoely86