Skip to content

[3.10] bpo-44461: Check early that a pdb target is valid for execution. (GH-27227) #27399

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 28, 2021

Conversation

jaraco
Copy link
Member

@jaraco jaraco commented Jul 28, 2021

https://bugs.python.org/issue44461

Automerge-Triggered-By: GH:jaraco

…n. (GH-27227)

* bpo-44461: Fix bug with pdb's handling of import error due to a package which does not have a __main__ module

* 📜🤖 Added by blurb_it.

* remove "else"

Co-authored-by: Jason R. Coombs <jaraco@jaraco.com>

* If running as a module, first check that it can run as a module. Alternate fix for bpo-44461.

Co-authored-by: Irit Katriel <iritkatriel@yahoo.com>
Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
Co-authored-by: Irit Katriel <1055913+iritkatriel@users.noreply.github.com>.
(cherry picked from commit ee03bad)

Co-authored-by: Jason R. Coombs <jaraco@jaraco.com>
@jaraco
Copy link
Member Author

jaraco commented Jul 28, 2021

Sorry for the late notice. Irit, if you would please have a look before Sunday so we can get this into the 3.10 RC? Normally, I'd just merge a backport, but since this is a transformed backport to span the refactor, I'd appreciate your review. I expect this change is functionally equivalent to the behavior of the change from which it was backported.

Copy link
Contributor

@ambv ambv left a comment

Choose a reason for hiding this comment

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

Confirmed this fixes the issue consistently with the original patch.

@miss-islington miss-islington merged commit 684eb5c into 3.10 Jul 28, 2021
@miss-islington miss-islington deleted the backport-ee03bad-3.10 branch July 28, 2021 13:04
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.

5 participants