Skip to content

Conversation

@encukou
Copy link
Member

@encukou encukou commented Nov 10, 2025

Copy link
Contributor

@DinoV DinoV left a comment

Choose a reason for hiding this comment

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

LGTM! I guess this means if the 1st find failed then the 2nd one will certainly fail too?

@encukou
Copy link
Member Author

encukou commented Nov 11, 2025

Yes.

At least that's the case for CPython's dynamic loading implementations (dynload_hpux.c, dynload_shlib.c, dynload_stub.c, dynload_win.c).
Of course, if someone ports to a different platform, they can add a dynload file (and set DYNLOADFILE in configure) -- and for that case this detail wasn't documented.

I added a comment here, and a TODO item to #140550 to add a note for DYNLOADFILE users to PEP 793 What's New entry.

@encukou encukou merged commit 799326b into python:main Nov 11, 2025
44 checks passed
@encukou encukou deleted the findfuncptr-error branch November 11, 2025 12:52
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.

2 participants