Skip to content

ENH: add hook and test for PyInstaller.#20745

Merged
charris merged 1 commit intonumpy:mainfrom
bwoodsend:pyinstaller-hook
Jan 24, 2022
Merged

ENH: add hook and test for PyInstaller.#20745
charris merged 1 commit intonumpy:mainfrom
bwoodsend:pyinstaller-hook

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!

@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 bwoodsend marked this pull request as draft January 11, 2022 18:35
@bwoodsend bwoodsend marked this pull request as ready for review January 12, 2022 22:42
@bwoodsend
Copy link
Contributor Author

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

@bwoodsend bwoodsend requested a review from rgommers January 12, 2022 22:44
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 #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