Skip to content

Add hook for zeroconf - #1024

Open
Mirochill wants to merge 1 commit into
pyinstaller:masterfrom
Mirochill:hook-840-zeroconf-hiddenimports
Open

Add hook for zeroconf#1024
Mirochill wants to merge 1 commit into
pyinstaller:masterfrom
Mirochill:hook-840-zeroconf-hiddenimports

Conversation

@Mirochill

@Mirochill Mirochill commented May 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add a standard hook for zeroconf that collects package submodules hidden behind cythonized wheels.
  • Add a frozen import test and pin zeroconf in the test-library requirements.
  • Add the required news fragment.

Closes #840.

Validation

@Mirochill
Mirochill marked this pull request as ready for review May 26, 2026 20:00
Comment thread news/1024.update.rst
@@ -0,0 +1 @@
Update ``zeroconf`` hook to collect cythonized package submodules.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The hook was added, not updated. So the news fragment should be 1024.new.rst, and the message itself should reflect that.

Comment thread tests/test_libraries.py
import zeroconf

print(zeroconf.__name__)
""")

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This test passes with PyInstaller >= 6.13 even without the hook being added - presumably thanks to pyinstaller/pyinstaller#9084 and .py files being available next to the cythonized extensions.

Which means this PR should be largely redundant, except for people who are sticking to older versions of PyInstaller for whatever reason.

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.

Module only partially copied - No module named 'zeroconf._utils.ipaddress'

2 participants