gh-137576: Fix for Basic REPL Showing Incorrect Code in Tracebacks with PYTHONSTARTUP by adqm · Pull Request #137625 · python/cpython · GitHub
Skip to content

gh-137576: Fix for Basic REPL Showing Incorrect Code in Tracebacks with PYTHONSTARTUP#137625

Merged
Eclips4 merged 8 commits into
python:mainfrom
adqm:basic_repl_error_reporting
Aug 14, 2025
Merged

gh-137576: Fix for Basic REPL Showing Incorrect Code in Tracebacks with PYTHONSTARTUP#137625
Eclips4 merged 8 commits into
python:mainfrom
adqm:basic_repl_error_reporting

Conversation

@adqm

@adqm adqm commented Aug 11, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

This patch attempts to fix #137576 by changing the way interactive_filename is computed for the basic REPL to make sure that it always begins with < and ends with > so that doesn't get an entry in linecache.cache (which was causing the source from PYTHONSTARTUP to be used instead of the interactive input when generating tracebacks).

Updated behavior:

$ echo "print('hello from PYTHONSTARTUP')" > /tmp/foo.py
$ PYTHON_BASIC_REPL=1 PYTHONSTARTUP=/tmp/foo.py ./python
Python 3.15.0a0 (heads/basic_repl_error_reporting:3d46e8feed, Aug 10 2025, 21:12:17) [GCC 12.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
hello from PYTHONSTARTUP
>>> 1/0
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
    1/0
    ~^~
ZeroDivisionError: division by zero

$ echo "def foo(x):  return 1/x" > /tmp/foo.py
$ PYTHON_BASIC_REPL=1 PYTHONSTARTUP=/tmp/foo.py ./python
Python 3.15.0a0 (heads/basic_repl_error_reporting:3d46e8feed, Aug 10 2025, 21:12:17) [GCC 12.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> def bar(x):
...     return foo(x)
... 
>>> bar(0)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
    bar(0)
    ~~~^^^
  File "<stdin>", line 2, in bar
    return foo(x)
  File "/tmp/foo.py", line 1, in foo
    def foo(x):  return 1/x
                        ~^~
ZeroDivisionError: division by zero

@bedevere-app bedevere-app Bot mentioned this pull request Aug 11, 2025
Closed
@adqm

adqm commented Aug 11, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Eclips4
Eclips4 approved these changes Aug 11, 2025
View reviewed changes

@Eclips4 Eclips4 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.

Thank you!
A couple of nits.

Comment thread Misc/NEWS.d/next/Core_and_Builtins/2025-08-10-21-34-12.gh-issue-137576.0ZicS-.rst Outdated
Comment thread Python/pythonrun.c Outdated
Comment thread Python/pythonrun.c
adqm and others added 2 commits August 11, 2025 14:45
@Eclips4

Eclips4 commented Aug 13, 2025

Copy link
Copy Markdown
Member

@adqm I'm going to merge this tomorrow.
Am I right that this reproducible on 3.13 and 3.14 (I don't have access to my machine now)?

@adqm

adqm commented Aug 13, 2025

Copy link
Copy Markdown
Contributor Author

Great, thanks!!

I just tested; the bug is indeed present on current 3.13 and 3.14 branches.

These patches apply cleanly to 3.14 as of right now but not to 3.13 (though I'm happy to make the necessary changes there).

@Eclips4 Eclips4 added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes labels Aug 13, 2025
@Eclips4 Eclips4 merged commit 04f8ef6 into python:main Aug 14, 2025
46 checks passed
@miss-islington-app

miss-islington-app Bot commented Aug 14, 2025

Copy link
Copy Markdown

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 14, 2025
@adqm @Eclips4 @miss-islington
pythongh-137576: Fix for Basic REPL showing incorrect code in traceba…
4f74d9f 
…cks with `PYTHONSTARTUP` (pythonGH-137625)

(cherry picked from commit 04f8ef6)

Co-authored-by: adam j hartz <hz@mit.edu>
Co-authored-by: Kirill Podoprigora <kirill.bast9@mail.ru>
@miss-islington-app

miss-islington-app Bot commented Aug 14, 2025

Copy link
Copy Markdown

Sorry, @adqm and @Eclips4, I could not cleanly backport this to 3.13 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 04f8ef663be7589def9f456a2024e1707e3408ea 3.13

@bedevere-app

bedevere-app Bot commented Aug 14, 2025

Copy link
Copy Markdown

GH-137771 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 Aug 14, 2025
@Eclips4

Eclips4 commented Aug 14, 2025

Copy link
Copy Markdown
Member

@adqm Could you please manually backport this to 3.13? Thanks in advance!

@serhiy-storchaka

serhiy-storchaka commented Aug 14, 2025

Copy link
Copy Markdown
Member

Reminder about backporting. @adqm @Eclips4

@bedevere-app

bedevere-app Bot commented Aug 14, 2025

Copy link
Copy Markdown

GH-137778 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.13 bugs and security fixes label Aug 14, 2025
@adqm

adqm commented Aug 14, 2025

Copy link
Copy Markdown
Contributor Author

Eclips4 added a commit that referenced this pull request Aug 15, 2025
@adqm @Eclips4
[3.13] gh-137576: Fix for Basic REPL showing incorrect code in traceb…
5131b8f 
…acks with `PYTHONSTARTUP` (GH-137625) (#137778)

(cherry picked from commit 04f8ef6)

Co-authored-by: Kirill Podoprigora <kirill.bast9@mail.ru>
Agent-Hellboy pushed a commit to Agent-Hellboy/cpython that referenced this pull request Aug 19, 2025
@adqm @Eclips4 @Agent-Hellboy
pythongh-137576: Fix for Basic REPL showing incorrect code in traceba…
9ddf280 
…cks with `PYTHONSTARTUP` (python#137625)

Co-authored-by: Kirill Podoprigora <kirill.bast9@mail.ru>
hugovk pushed a commit that referenced this pull request Aug 27, 2025
@miss-islington @adqm @Eclips4
[3.14] gh-137576: Fix for Basic REPL showing incorrect code in traceb…
6d30edb 
…acks with `PYTHONSTARTUP` (GH-137625) (#137771)

Co-authored-by: adam j hartz <hz@mit.edu>
Co-authored-by: Kirill Podoprigora <kirill.bast9@mail.ru>
kumaraditya303 pushed a commit to miss-islington/cpython that referenced this pull request Sep 9, 2025
@miss-islington @adqm
@Eclips4 @kumaraditya303
[3.14] pythongh-137576: Fix for Basic REPL showing incorrect code in …
7f13b60 
…tracebacks with `PYTHONSTARTUP` (pythonGH-137625) (python#137771)

Co-authored-by: adam j hartz <hz@mit.edu>
Co-authored-by: Kirill Podoprigora <kirill.bast9@mail.ru>
@picnixz picnixz mentioned this pull request Sep 22, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Eclips4 Eclips4 Eclips4 approved these changes
@ericsnowcurrently ericsnowcurrently Awaiting requested review from ericsnowcurrently ericsnowcurrently is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

traceback shows wrong source code in basic REPL when PYTHONSTARTUP is set

3 participants

@adqm @Eclips4 @serhiy-storchaka