8000 BUG: fix f2py shebang line for bdist wheel, egg by matthew-brett · Pull Request #5815 · numpy/numpy · GitHub
[go: up one dir, main page]

Skip to content

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

Merged
merged 2 commits into from
Apr 30, 2015

Conversation

matthew-brett
Copy link
Contributor

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 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 gh-5812.

should be ``#!python`` rather than ``#!`` followed by the contents of
``sys.executable``.
"""
import IPython; IPython.embed()
Copy link
Member

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

Copy link
Contributor Author

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.
@rgommers
Copy link
Member

It doesn't look that hacky to me. #!python should work. I think that the Windows installers have the same issue (haven't checked, but has to be the case). We create those with bdist_wininst, so would be good to add that string as well. And while we're at it, bdist_mpkg for the OS X dmg installers.

@rgommers rgommers added this to the 1.10.0 release milestone Apr 29, 2015
Add other binary distribution formats to list of build commands that
should generate !python shebang lines.
@matthew-brett
Copy link
Contributor Author

It would be good to test this is working - here's a first pass : #5817

@matthew-brett
Copy link
Contributor Author

#5817 now working I believe - so if you merge this PR, it might be worth considering #5817 to test this fix.

charris added a commit that referenced this pull request Apr 30, 2015
BUG: fix f2py shebang line for bdist wheel, egg
@charris charris merged commit 38cb7aa into numpy:master Apr 30, 2015
@charris
Copy link
Member
charris commented Apr 30, 2015

Merged, thanks Matthew.

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

Successfully merging this pull request may close these issues.

f2py: incorrect call to Python interpreter
3 participants
0