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

Should we modify lib2to3? #7

Closed
liamcurry opened this issue Apr 19, 2013 · 4 comments
Closed

Should we modify lib2to3? #7

liamcurry opened this issue Apr 19, 2013 · 4 comments
Labels

Comments

@liamcurry
Copy link
Owner

Since we are already bundling lib2to3 with this package, do you think we should modify it, or leave it as is? There's a lot of code in there that could use some cleaning up. Also, if we're looking to increase performance then we'll probably have to do this anyway.

@dustinmm80
Copy link
Collaborator

I'd say since we're vendorizing it, we might as well improve it. If it was being continually released, we would have to worry about compatibility, but since this lib isn't changing on 2.x I'd say we can fix it up.

@myint
Copy link
Collaborator

myint commented Apr 19, 2013

If necessary, then sure. Though we should keep in mind that things like python-modernize manage to do everything they need with the default lib2to3 (without vendorizing/modifying). And it even works on Python 2.6, 2.7, 3.2, and 3.3. (See the unwanted_fixes code for example.)

@liamcurry
Copy link
Owner Author

@myint True, but the goals of these projects seems to be different. python-modernize and lib2to3 have a large emphasis on automatic refactoring of existing code bases, whereas py3kwarn is basically just a syntax checker used to help developers write forward-compatible code.

@myint
Copy link
Collaborator

myint commented Apr 19, 2013

Right. I only mention python-modernize so you guys know that you can get pretty far (in terms of both functionality and Python compatibility) with the default lib2to3.

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

3 participants