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
Improve error finding #4001
Improve error finding #4001
Conversation
index += 1 | ||
|
||
|
||
def parse(self, stream, context=6, jobs=None): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
too many blank lines (2)
warning_exceptions = compile(_warning_exceptions) | ||
file_line_matches = compile(_file_line_matches) | ||
|
||
matcher, args = _match, [] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
local variable 'args' is assigned to but never used
error_exceptions = compile(_error_exceptions) | ||
warning_matches = compile(_warning_matches) | ||
warning_exceptions = compile(_warning_exceptions) | ||
file_line_matches = compile(_file_line_matches) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
multiple spaces before operator
|
||
error_matches = compile(_error_matches) | ||
error_exceptions = compile(_error_exceptions) | ||
warning_matches = compile(_warning_matches) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
multiple spaces before operator
return [re.compile(regex) for regex in regex_array] | ||
|
||
error_matches = compile(_error_matches) | ||
error_exceptions = compile(_error_exceptions) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
multiple spaces before operator
def compile(regex_array): | ||
return [re.compile(regex) for regex in regex_array] | ||
|
||
error_matches = compile(_error_matches) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
multiple spaces before operator
Closing this, as we'll look to implement some method to cover this as part of the EB 5.0 work. |
(created using
eb --new-pr
)This is a proof of concept. This is based off the discussion from EUM on this and I've implemented this in a way that allows others to test the messages it highlights.
This:
ctest_log_parser.py
from spack and puts it in a newexternal
directorystdouterr
for each stage and: