Skip to content

gh-138122: Split Modules/_remote_debugging_module.c into multiple files - #141934

Merged
pablogsal merged 1 commit into
python:mainfrom
pablogsal:gh-138122-refactor
Nov 25, 2025
Merged

gh-138122: Split Modules/_remote_debugging_module.c into multiple files#141934
pablogsal merged 1 commit into
python:mainfrom
pablogsal:gh-138122-refactor

Conversation

@pablogsal

@pablogsalpablogsal commented Nov 25, 2025•
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

@pablogsalpablogsal changed the title gh-1381228: Split Modules/_remote_debugging_module.c into multiple filesgh-138122: Split Modules/_remote_debugging_module.c into multiple filesNov 25, 2025
@bedevere-appbedevere-appBot mentioned this pull request Nov 25, 2025
Open
@pablogsalpablogsal added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Nov 25, 2025
@bedevere-bot

bedevere-bot commented Nov 25, 2025

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @pablogsal for commit dfd0056 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F141934%2Fmerge

If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again.

@bedevere-botbedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Nov 25, 2025
@pablogsalpablogsal added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Nov 25, 2025
@bedevere-bot

bedevere-bot commented Nov 25, 2025

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @pablogsal for commit 0b2efa0 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F141934%2Fmerge

If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again.

@bedevere-botbedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Nov 25, 2025
@pablogsal

pablogsal commented Nov 25, 2025

Copy link
Copy Markdown
MemberAuthor

There is no new code here, just moving functions into new files as this file was getting unmanageable :)

emmatyping
emmatyping reviewed Nov 25, 2025
View reviewed changes

@emmatypingemmatyping left a comment

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.

I didn't look at the splitting of the files, but the build system changes look good.

@pablogsal

pablogsal commented Nov 25, 2025•
edited
Loading

Copy link
Copy Markdown
MemberAuthor

Thanks a lot for the review of the build system files @emmatyping 🖤

@pablogsal
pablogsal merged commit d07d3a3 into python:mainNov 25, 2025
101 checks passed
@pablogsal
pablogsal deleted the gh-138122-refactor branch November 25, 2025 12:51
StanFromIreland pushed a commit to StanFromIreland/cpython that referenced this pull request Dec 6, 2025
@pablogsal@StanFromIreland
pythongh-138122: Split Modules/_remote_debugging_module.c into multip…
76a88da
…le files (python#141934)
gh-1381228: Split Modules/_remote_debugging_module.c into multiple files
ashm-dev pushed a commit to ashm-dev/cpython that referenced this pull request Dec 8, 2025
@pablogsal@ashm-dev
pythongh-138122: Split Modules/_remote_debugging_module.c into multip…
ead61f0
…le files (python#141934)
gh-1381228: Split Modules/_remote_debugging_module.c into multiple files
jjhelmus added a commit to jjhelmus/python-build-standalone that referenced this pull request Dec 17, 2025
@jjhelmus
_remote_debugging module sources are split in 3.15
bc1ee90
The _remote_debugging module was split into multiple files in:
python/cpython#141934
Update the extensions-modules.yml file to reflect this change.
@brianschubertbrianschubert mentioned this pull request Mar 24, 2026
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@emmatypingemmatypingemmatyping left review comments
@ambvambvAwaiting requested review from ambv
@1st11st1Awaiting requested review from 1st1
@erlend-aaslanderlend-aaslandAwaiting requested review from erlend-aaslanderlend-aasland is a code owner
@AA-TurnerAA-TurnerAwaiting requested review from AA-TurnerAA-Turner is a code owner
@corona10corona10Awaiting requested review from corona10corona10 is a code owner

Assignees

No one assigned

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@pablogsal@bedevere-bot@emmatyping