-
-
Notifications
You must be signed in to change notification settings - Fork 10.9k
BUG: fix f2py shebang line for bdist wheel, egg #5815
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
Conversation
should be ``#!python`` rather than ``#!`` followed by the contents of | ||
``sys.executable``. | ||
""" | ||
import IPython; IPython.embed() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
left-over line from debugging here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh dear - thanks for catching that.
Command `bdist_wheel` was generating a shebang line for f2py that uses the Python path for the building Python. If we are building a wheel or an egg, use the generic `#!python` shebang line for the f2py script instead, which setuptools will modify at install time. Closes numpygh-5812.
a90a682
to
fee5ec4
Compare
It doesn't look that hacky to me. |
Add other binary distribution formats to list of build commands that should generate !python shebang lines.
It would be good to test this is working - here's a first pass : #5817 |
BUG: fix f2py shebang line for bdist wheel, egg
Merged, thanks Matthew. |
This is a painful hack, but I don't know a better way. So, I put it forward
for improvement by y'all.
Command
bdist_wheel
was generating a shebang line for f2py that uses thePython path for the building Python. If we are building a wheel or an egg,
use the generic
#!python
shebang line for the f2py script instead, whichsetuptools will modify at install time.
Closes gh-5812.