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

check-software: fix undefined variable traceback #2683

Merged
merged 1 commit into from
May 31, 2018

Conversation

sadielbartholomew
Copy link
Collaborator

Another bug has been found in check-software. When a Python module import fails within the check_py_module_ver() function, module_version does not get defined, leading to a traceback when trying to compare that variable with the (un)specified minimum version & report the result.

@sadielbartholomew sadielbartholomew added the bug Something is wrong :( label May 29, 2018
@sadielbartholomew sadielbartholomew added this to the next release milestone May 29, 2018
@sadielbartholomew sadielbartholomew self-assigned this May 29, 2018
@sadielbartholomew
Copy link
Collaborator Author

sadielbartholomew commented May 29, 2018

(Ignore open/close of PR made by mistake).

@hjoliver
Copy link
Member

A user hit this bug today, due to not have pygraphviz installed. Nice to be able to report it has been fixed already!

Copy link
Member

@hjoliver hjoliver left a comment

Choose a reason for hiding this comment

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

Tested as working.

@hjoliver hjoliver merged commit dc89b7f into cylc:master May 31, 2018
@hjoliver
Copy link
Member

(PR needed to 'next maintenance release' too)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is wrong :(
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants