DOC: Remove and Update out of date Docker Image issue with #61511 #61550
+25
−25
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jacksnnn
commented
Jun 4, 2025
jacksnnn
commented
- Addresses & closes DOC: Docker image provided on "Debugging C extensions" is out of date #61511
- All code checks passed.
|
@WillAyd do you mind having a look? I think we should also remove the container if it's not maintained, not only the docs. |
WillAyd
reviewed
Jun 4, 2025
| +Using ``cygdb`` directly (recommended) | ||
| +-------------------------------------- | ||
| + | ||
| +The previously advertised Docker image (``pandas/pandas-debug``) is **no longer |
There was a problem hiding this comment.
Let's remove any references to the previously documented approach and just keep this as reflecting the current best practice
| + <debugging_c_extensions>`). | ||
| +2. Install the helpers:: | ||
| + | ||
| + pip install cython cython-gdb |
There was a problem hiding this comment.
Do we need this? Shouldn't the environment users have already set up for development already provide this?
| +variables with the commands documented in the `Cython debugging guide | ||
| +<https://docs.cython.org/en/latest/src/userguide/debugging.html#using-the-debugger>`_. | ||
| + | ||
| +If you need a full system debugger, ensure *gdb* (or *lldb* on macOS) is |
There was a problem hiding this comment.
Aren't these required to use cygdb in the first place?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.