[Lldb-commits] [lldb] [lldb-dap] Change the launch sequence (PR #138219)

[Pavel Labath via lldb-commits]

labath wrote:

I think I've found a(nother) problem with this change. One of the arguments of the launch request is `sourceMap`, which sets the `target.source-map` setting. The setting is necessary to correctly resolve file+line breakpoints in case the file's path on the host system does not match the path from the debug info. By deferring the processing of this request, we're effectively ignoring this setting as all of the initial breakpoint requests (which are not deferred) will not see the intended value. This is kind of a Big Deal(tm) for us since all of our binaries are built in the cloud.

I see this has been kind of reverted in #140331. I suspect that fixes this use case (checking that is my next step), but I'm writing this to let you know of the problem.

https://github.com/llvm/llvm-project/pull/138219