Skip to content
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

pdb should check if the executing file is modified and warn users #111051

Closed
gaogaotiantian opened this issue Oct 19, 2023 · 2 comments
Closed

pdb should check if the executing file is modified and warn users #111051

gaogaotiantian opened this issue Oct 19, 2023 · 2 comments
Labels
stdlib Python modules in the Lib dir type-feature A feature request or enhancement

Comments

@gaogaotiantian
Copy link
Member

gaogaotiantian commented Oct 19, 2023

Feature or enhancement

Proposal:

It's possible that the file is modified during debugging, which would definitely confuse users - the source will not match the execution anymore.

There's no perfect solution under such circunstances, but we should at least warn users that this is happening.

Has this already been discussed elsewhere?

This is a minor feature, which does not need previous discussion elsewhere

Links to previous discussion of this feature:

No response

Linked PRs

@ambv
Copy link
Contributor

ambv commented Jul 18, 2024

I often have an issue with the test for this when running on a RAM disk formatted with HFS+. That file system only has 1-second mtime resolution, so the test often doesn't register an mtime change and intermittently fails. Fix incoming.

ambv added a commit to ambv/cpython that referenced this issue Jul 18, 2024
ambv added a commit to ambv/cpython that referenced this issue Jul 22, 2024
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Jul 22, 2024
…w-resolution mtime (pythonGH-121959)

(cherry picked from commit cad11a2)

Co-authored-by: Łukasz Langa <lukasz@langa.pl>
serhiy-storchaka pushed a commit that referenced this issue Aug 12, 2024
…ow-resolution mtime (GH-121959) (GH-122114)

(cherry picked from commit cad11a2)

Co-authored-by: Łukasz Langa <lukasz@langa.pl>
@gaogaotiantian
Copy link
Member Author

Hi @ambv , is this fixed now? Can we close this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stdlib Python modules in the Lib dir type-feature A feature request or enhancement
Projects
None yet
Development

No branches or pull requests

3 participants