Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork32.1k
Fix definition of_Py_RemoteDebug_
symbols for static linking#135146
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
zanieb commentedJun 4, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
I'm happy to include a news entry and issue if desired, we chatted about this problem briefly in Discord and I figured I'd just quickly post the patch. |
CI has some compiler warnings
|
Yeah this file is included in two places but they don't use all the functions. We must silence these warnings because they are false positives |
No need. This is not a user visible change and it doesn't really qualify as a bug fix (depends how you look at it). I am happy either way basically |
@pablogsal I've silenced the warnings. |
Hummm, I still see the warnings in GitHub in thehttps://github.com/python/cpython/pull/135146/files page. Is this expected? |
Yeah I think so? I'm not familiar with the warnings tooling, but I believe I've just followed the "standard" way to silence new warnings in a file. If there's another way, we'll need to hear from someone who's an owner of such things :) |
Uh oh!
There was an error while loading.Please reload this page.
As noted at
cpython/Python/remote_debug.h
Line 9 in01bb53d
python-build-standalone
), e.g.: