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.

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

Ignore some error codes that can be given by modulecmd, these can happen... #609

Merged
merged 2 commits into from Apr 25, 2013

Conversation

JensTimmerman
Copy link
Contributor

... withsymlinks with which point to a directory where we have no read persmissions

This should be allowed, since this system of pointing to unreadable directories allows us to hide modules where a user has no acces to anyway
but they will automatically show up for users who have the right permissions.

…pen withsymlinks with which point to a directory where we have no read persmissions

 This should be allowed, since this system of pointing to unreadable directories allows us to hide modules where a user has no acces to anyway
 but they will automatically show up for users who have the right permissions.
@@ -57,7 +57,10 @@
# matches whitespace and module-listing headers
'whitespace': re.compile(r"^\s*$|^(-+).*(-+)$"),
# matches errors such as "cmdTrace.c(713):ERROR:104: 'asdfasdf' is an unrecognized subcommand"
'error': re.compile(r"^\S+:(?P<level>\w+):(?P<code>\d+):\s+(?P<msg>.*)$"),
## following erros should not be matches, they are considered warnings
Copy link
Member

Choose a reason for hiding this comment

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

typo: 'matches' -> 'matched'

@JensTimmerman
Copy link
Contributor Author

fixed

boegel added a commit that referenced this pull request Apr 25, 2013
Ignore some error codes that can be given by modulecmd, these can happen...
@boegel boegel merged commit 8354361 into easybuilders:develop Apr 25, 2013
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.

None yet

2 participants