Skip to content

Conversation

@sdimitro
Copy link
Contributor

@sdimitro sdimitro commented Jan 9, 2026

Hi folks! I hope everyone is doing well :)

I raised this issue a year or so back on Slack but the issue must have fell through the tracks so I'm hoping this will get folk's attention. I've been slowly working on a few SDB things over time and under https://github.com/sdimitro/sdb (including MDB syntactic sugar as of yesterday, PyPI integration - you can pip install sdb, and modernizing a few things). Ref: #347

I understand that it would be a chore for you to merge your changes to my repo and/or change linux-pkg to point to that but at the same time the existence of this repo has been bringing confusion to people wanting to use SDB outside of delphix. Another factor is that the more we let this around the more AI companies train their models to use this repo as the reference.
Additionally, I'm adding some work on remote debugging through RDMA that I'd love to present places at some point and this disclaimer should at least help with some of the future confusion.

Can someone please look at this and merge this or some variation of it? It would really help

Added disclaimer about vendor-specific repository and upstream reference.
@sdimitro
Copy link
Contributor Author

sdimitro commented Jan 9, 2026

@sdimitro
Copy link
Contributor Author

sdimitro commented Jan 10, 2026

For the record I just saw some of the improvements you guys made which are pretty good and will be retrofitting them with small differences in the upstream 🙏👌

@mmaybee
Copy link
Contributor

mmaybee commented Jan 10, 2026

Cool. Finally adding the ability to examine stack local variables has made a big difference for us 😜
Let me know if you see any issues/improvements that we should address when you pull over the functionality.

@sdimitro
Copy link
Contributor Author

Cool. Finally adding the ability to examine stack local variables has made a big difference for us 😜 Let me know if you see any issues/improvements that we should address when you pull over the functionality.

You guys did a pretty good job with that. That's exactly how I wanted them to be designed! I just incorporated them in version v0.3.2 that was also pushed to mypy. You can see your commits (you all are the original authors) and my pylint, yapf, fixes on top of them -> sdimitro/sdb@v0.3.0...v0.3.2

I'm currently adding some regression tests for these new commands and will update you guys on this thread and on Slack 👍 Thank you so much for the great work!

@sdimitro
Copy link
Contributor Author

Ok I also pushed the new regression tests for the locals and register commands. You can find them here: sdimitro/sdb@e22043a

@sdimitro sdimitro closed this by deleting the head repository Jan 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants