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

ENH: add hook and test for PyInstaller. #20745

Merged
merged 1 commit into from Jan 24, 2022
Merged

Conversation

bwoodsend
Copy link
Contributor

Adding this special hook file tells PyInstaller what files a self contained NumPy application needs to run. A test is encluded to verify that the hook still finds all necessary files.

Closes #17184.

Copy link
Member

@rgommers rgommers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks really good, thanks @bwoodsend!

numpy/tests/test_pyinstaller.py Outdated Show resolved Hide resolved
numpy/hook-numpy.py Outdated Show resolved Hide resolved
numpy/hook-numpy.py Outdated Show resolved Hide resolved
numpy/hook-numpy.py Outdated Show resolved Hide resolved
numpy/tests/test_public_api.py Outdated Show resolved Hide resolved
@bwoodsend bwoodsend force-pushed the pyinstaller-hook branch 3 times, most recently from b8acf2a to 252ea91 Compare January 8, 2022 13:55
@bwoodsend
Copy link
Contributor Author

The two travis failures will disappear when PyInstaller next releases (which was supposed to be this morning) shipping pre-compiled wheels for those architectures. Something unpleasant appears to have knocked out installing cygwin. Hopefully egor-tensin/setup-cygwin will mop it up quickly. What's wrong with Windows 32bit on Azure is a mystery to me.

@bwoodsend
Copy link
Contributor Author

PyInstaller's ppc64le and s390x wheels have finally hit PyPI so CI can pass now.

Adding this special hook file tells PyInstaller what files a self contained
NumPy application needs to run. A test is included to verify that the hook still
finds all necessary files.

Closes numpy#17184.
@bwoodsend
Copy link
Contributor Author

That's this rebased so as to include #20831.

@charris charris merged commit 6218e76 into numpy:main Jan 24, 2022
@charris
Copy link
Member

charris commented Jan 24, 2022

Thanks @bwoodsend .

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.

Provide hooks freezing with for PyInstaller?
3 participants