gh-102809: No longer mention Misc/gdbinit in the code

[johnslavik]

Misc/gdbinit was removed in GH-102854 (3.12.0a7).

• Issue: What to do with Misc/gdbinit? #102809

[StanFromIreland]

Or fix it like this one was done?

cpython/Doc/howto/gdb_helpers.rst

Line 40 in 63cc125

- GDB 7 or later. (For earlier versions of GDB, see ``Misc/gdbinit`` in the

[johnslavik]

Or fix it like this one was done?

cpython/Doc/howto/gdb_helpers.rst

Line 40 in 63cc125

- GDB 7 or later. (For earlier versions of GDB, see ``Misc/gdbinit`` in the

I'm not sure. This comment remains on 3.10, 3.11 and 3.12 branches, would including Misc/gdbinit again be helpful?
Or do you mean mentioning GDB 7?
Or do you have in mind some other specific idea of improving this comment?

We could direct the reader to Doc/howto/gdb_helpers.rst maybe?

[StanFromIreland]

Oh I only have the one line shown, I was referring to: "see Misc/gdbinit in the sources of Python 3.11 or earlier."

[johnslavik]

Thanks. Personally, I don't find it necessary, because the 3.11 and earlier branches still will include the mention of Misc/gdbinit after we merge this.

[StanFromIreland]

Why do those branches matter? The comment is for people on the branches where the file doesn't exist, no?

[johnslavik]

Why do those branches matter? The comment is for people on the branches where the file doesn't exist, no?

I was thinking that mentioning Misc/gdbinit would not be useful to people on branches that don't contain the file, because they wouldn't need nor be able to use the file.

If they needed to debug 3.11, they would check it out and the comment would be there.
Unless I am missing something?

[johnslavik]

Do you have an example of when this is useful for someone debugging in the 3.13-3.15 branches only?

[ZeroIntensity]

Yeah, I agree with Bartosz here. The mention in the docs was added afterMisc/gdbinit was removed, so I think it's intentional to keep that one.

[vstinner]

LGTM

[miss-islington-app]

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

[bedevere-app]

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

[bedevere-app]

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

[vstinner]

Merged, thanks for this cleanup.

[johnslavik]

@vstinner, thanks for the review and for linking the issue!