Skip to content

Fix undefined name "sys" #299

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

Merged
merged 1 commit into from
Jul 4, 2015
Merged

Fix undefined name "sys" #299

merged 1 commit into from
Jul 4, 2015

Conversation

mnaberez
Copy link
Contributor

@mnaberez mnaberez commented Jul 3, 2015

sys.stderr is used in an except clause of fix_source_eol() but it will cause NameError: name 'sys' is not defined because the sys module is never imported.

@cdunn2001
Copy link
Contributor

AppVeyor is down, but this is simple.

cdunn2001 added a commit that referenced this pull request Jul 4, 2015
@cdunn2001 cdunn2001 merged commit bca0eff into open-source-parsers:master Jul 4, 2015
cdunn2001 pushed a commit to cdunn2001/jsoncpp that referenced this pull request Jul 11, 2015
Same as open-source-parsers#299 (bca0eff), but an earlier
commit needed to be rebased.
cdunn2001 added a commit that referenced this pull request Jul 11, 2015
Fix undefined name "sys"

Same as #299 (bca0eff), but an earlier commit needed to be rebased.
cdunn2001 pushed a commit to cdunn2001/jsoncpp that referenced this pull request Jul 11, 2015
Same as open-source-parsers#299 (bca0eff), but an earlier
commit needed to be rebased.
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.

2 participants