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

epylint doesn't work on Python 3 #7

Closed
pylint-bot opened this issue Apr 28, 2013 · 4 comments
Closed

epylint doesn't work on Python 3 #7

pylint-bot opened this issue Apr 28, 2013 · 4 comments
Labels

Comments

@pylint-bot
Copy link

Originally reported by: BitBucket: jan_samohyl


Running epylint on Python 3.2 fails with:

Traceback (most recent call last):
File "/usr/local/bin/epylint", line 3, in
epylint.Run()
File "/usr/local/lib/python3.2/dist-packages/pylint/epylint.py", line 161, in Run
sys.exit(lint(sys.argv[1]))
File "/usr/local/lib/python3.2/dist-packages/pylint/epylint.py", line 96, in lint
if line.startswith("No config file found"):
TypeError: startswith first arg must be bytes or a tuple of bytes, not str

I am attaching a small fix that seems to work for me.


@pylint-bot
Copy link
Author

Original comment by Nikolaus Rath (BitBucket: nikratio, GitHub: @Nikratio?):


I think a better fix is to add universal_newlines = True to the Popen call.

@pylint-bot
Copy link
Author

Original comment by Sylvain Thénault (BitBucket: sthenault, GitHub: @sthenault?):


I've commited a cset to use universal_newlines on epylint Popen calls.

@pylint-bot
Copy link
Author

Original comment by Sylvain Thénault (BitBucket: sthenault, GitHub: @sthenault?):


fix epylint w/ python3. Closes issue #7

@pylint-bot
Copy link
Author

Original comment by Sylvain Thénault (BitBucket: sthenault, GitHub: @sthenault?):


Removing version: 1.0 (automated comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant