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

Minor import bug #54

Open
danijel3 opened this issue Aug 19, 2016 · 1 comment
Open

Minor import bug #54

danijel3 opened this issue Aug 19, 2016 · 1 comment

Comments

@danijel3
Copy link

Don't feel like making a pull requesrt for a single line of code.

In scripts/internal/pocolm_common.py:55, CalledProcessError is used without being properly imported. This error occurs only if subprocess.check_output fails.

Solution:
either change that to read subprocess.CalledProcessError or add from subprocess import CalledProcessError at the beginning of the file.

@danpovey
Copy link
Owner

+keli78, can you please make a PR for this?

On Fri, Aug 19, 2016 at 5:57 AM, Danijel Koržinek [email protected]
wrote:

Don't feel like making a pull requesrt for a single line of code.

In scripts/internal/pocolm_common.py:55, CalledProcessError is used
without being properly imported. This error occurs only if
subprocess.check_output fails.

Solution:
either change that to read subprocess.CalledProcessError or add from
subprocess import CalledProcessError at the beginning of the file.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#54, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADJVu5yim30uIGtqa90wPYidSf9T22ANks5qhYwmgaJpZM4JoXgz
.

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

No branches or pull requests

2 participants