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

Is upload.py compatible with python 3.3.3? #486

Open
GoogleCodeExporter opened this issue Mar 28, 2015 · 3 comments
Open

Is upload.py compatible with python 3.3.3? #486

GoogleCodeExporter opened this issue Mar 28, 2015 · 3 comments

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. run upload.py

What is the expected output? What do you see instead?
I'm not sure as I get the following error:

# python \apps\bin\upload.py
  File "\apps\bin\upload.py", line 190
    except IOError, e:

What browser are you using?  What version? On what operating system?
Windows 7 64-bit
Python 3.3.3

At what URL are you accessing Rietveld?  
https://codereview.appspot.com/

Please provide any additional information below.

Original issue reported on code.google.com by manu.st...@gmail.com on 27 Mar 2014 at 9:17

@GoogleCodeExporter
Copy link
Author

The output I got was truncated:

# python  \apps\bin\upload.py
  File "\apps\bin\upload.py", line 190
    except IOError, e:
                  ^
SyntaxError: invalid syntax

Original comment by manu.st...@gmail.com on 27 Mar 2014 at 9:18

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

upload.py currently do not support python 3.x. If can submit changes that keep 
it compatible with python 2.7, I'll be happy to review these.

Original comment by maruel@chromium.org on 28 Mar 2014 at 12:10

  • Added labels: ****
  • Removed labels: ****

@pdbj
Copy link

pdbj commented Jun 3, 2015

Continuation of #247

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

No branches or pull requests

2 participants