-
Notifications
You must be signed in to change notification settings - Fork 144
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
Allow server to configure its root #1130
Comments
If paths are ephemeral and you can't pre-configure them to apply path mappings, maybe you could use an API which would collect the paths and runtime and set the mapping for the debugger? See: #743 (comment) for how to do that. |
I was also thinking that maybe having the path mapping allow for environment variables set in the server could potentially be a solution here too (so, you could have an environment variable resolved in the server as a part of the path mapping resolution) -- note that this isn't implemented at this point. |
So that'd be relevant here? debugpy/src/debugpy/_vendored/pydevd/_pydevd_bundle/pydevd_process_net_command_json.py Line 322 in 4067700
|
Yes. |
Ah hey that works, thanks! I'd still advocate for a env var, as thats much easier. |
OK I actually did something slightly different:
I didn't want to tell our users to omit |
In the Pants project we're facing an issue where we can't set breakpoints on files when running some code, and it boils down to the
remoteRoot
being CWD but code is running in a tempr sandbox.To give an overview of the workflow here:
/tmp/...
with all the code laid outdebugpy
) and waits for connection"."
which isn't the tempdir breakpoints relative to the code root aren't loadedYou could say "OK, the user should be setting the
pathMapping
", which would be annoying at the least, but is ultimately impossible as the sandboxes (the tempdir) is ephemeral, fresh, and distinct per run.I suspect what I really want is to allow the server to (somehow) specify it's "root" when being invoked (which in this case would be the sandbox tempdir)
The text was updated successfully, but these errors were encountered: