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

Fix shebang lines in python scripts #622

Closed
wants to merge 1 commit into from
Closed

Fix shebang lines in python scripts #622

wants to merge 1 commit into from

Conversation

panzarino
Copy link
Contributor

The statement print "..." can only be used in python 2, not python 3 (python 3 uses print()). This makes sure that the files are executed with python 2 and not python 3 because some users may have python 3 as their default.

@modocache
Copy link
Contributor

Alternatively, we could keep the shebangs as they are, and from __future__ import print_function. But I imagine these scripts are not Python 3 compatible in other ways...?

@modocache
Copy link
Contributor

utils/cmpcodesize also contains two scripts that use the generic python shebang. Should those be included in this pull request?

@gribozavr
Copy link
Contributor

Unfortunately this approach does not work on OS X, there's no python2.

Instead, I would appreciate review help with #90.

@gribozavr gribozavr closed this Dec 17, 2015
@panzarino
Copy link
Contributor Author

Sorry, I'm running on linux. My bad.

@panzarino panzarino deleted the fix-shebang branch December 17, 2015 18:28
freak4pc pushed a commit to freak4pc/swift that referenced this pull request Sep 28, 2022
Add swift-protobuf-plugin-example to test SwiftPM build tool plugin functionality
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.

3 participants