Skip to content

Conversation

Zheaoli
Copy link
Contributor

@Zheaoli Zheaoli commented Jun 15, 2025

Zheaoli added 2 commits June 16, 2025 02:31
Signed-off-by: Manjusaka <[email protected]>
Zheaoli added 2 commits June 16, 2025 03:09
Signed-off-by: Manjusaka <[email protected]>
Signed-off-by: Manjusaka <[email protected]>
@Zheaoli
Copy link
Contributor Author

Zheaoli commented Jun 16, 2025

@vstinner @sobolevn @pablogsal I have updated the test. PTAL when you got time

Zheaoli added 3 commits June 17, 2025 22:44
Signed-off-by: Manjusaka <[email protected]>
Signed-off-by: Manjusaka <[email protected]>
Signed-off-by: Manjusaka <[email protected]>
@Zheaoli
Copy link
Contributor Author

Zheaoli commented Jun 17, 2025

@vstinner I have fix the review idea. PTAL when you got time. BTW https://github.com/python/cpython/actions/runs/15710806342/job/44268135397?pr=135544 seems unrelated failed. Would you mind rerunning it for me?

@Zheaoli Zheaoli requested a review from vstinner June 17, 2025 15:21
Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

Thanks for the update, I like the updated code. Another review.

Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM.

@pablogsal: what do you think?

Signed-off-by: Manjusaka <[email protected]>
@Zheaoli
Copy link
Contributor Author

Zheaoli commented Jun 18, 2025

@vstinner sorry about this, I update a little bit of code. PTAL

Zheaoli added 2 commits June 18, 2025 22:38
Signed-off-by: Manjusaka <[email protected]>
Signed-off-by: Manjusaka <[email protected]>
Zheaoli added 2 commits June 19, 2025 02:42
Signed-off-by: Manjusaka <[email protected]>
Signed-off-by: Manjusaka <[email protected]>
@Zheaoli Zheaoli requested a review from zooba June 18, 2025 18:44
@pablogsal
Copy link
Member

LGTM.

@pablogsal: what do you think?

I am ok with this 👍

Signed-off-by: Manjusaka <[email protected]>
@ericsnowcurrently ericsnowcurrently removed their request for review June 19, 2025 15:00
@Zheaoli
Copy link
Contributor Author

Zheaoli commented Jun 19, 2025

@zooba I have updated my PR. PTAL

@zooba
Copy link
Member

zooba commented Jun 19, 2025

LGTM, and I don't think the changes are significant since others signed off, so I'll merge.

@zooba zooba merged commit 1ddfe59 into python:main Jun 19, 2025
40 checks passed
@zooba zooba added the needs backport to 3.14 bugs and security fixes label Jun 19, 2025
@miss-islington-app
Copy link

Thanks @Zheaoli for the PR, and @zooba for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 19, 2025
…c is called (pythonGH-135544)

(cherry picked from commit 1ddfe59)

Co-authored-by: Nadeshiko Manju <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Jun 19, 2025

GH-135732 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label Jun 19, 2025
zooba pushed a commit that referenced this pull request Jun 19, 2025
…alled (GH-135544)

(cherry picked from commit 1ddfe59)

Co-authored-by: Nadeshiko Manju <[email protected]>
lkollar pushed a commit to lkollar/cpython that referenced this pull request Jun 19, 2025
@Zheaoli Zheaoli deleted the manjusaka/fix135543 branch June 19, 2025 21:53
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
taegyunkim pushed a commit to taegyunkim/cpython that referenced this pull request Aug 4, 2025
Agent-Hellboy pushed a commit to Agent-Hellboy/cpython that referenced this pull request Aug 19, 2025
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.

Add new audit event for sys.remote_exec()

5 participants