-
Notifications
You must be signed in to change notification settings - Fork 305
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
feat(clion): Add working directory to CLion Run configurations #6698
base: master
Are you sure you want to change the base?
Conversation
Unfortunately the setting doesn't work with gdbserver wrapper. I think it could be fixed here intellij/clwb/src/com/google/idea/blaze/clwb/run/BlazeGDBServerProvider.java Lines 100 to 115 in 9343e7e
maybe we could just add |
Indeed since edit: unfortunately cwd is not inherited, so |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the PR! Could you please update BlazeGDBServerProvider
as @tpasternak suggested?
Checklist
Please note that the maintainers will not be reviewing this change until all checkboxes are ticked. See
the Contributions section in the README for more
details.
Discussion thread for this change
Issue number: N/A
Description of this change
In CMake CLion, one has the option of setting a working directory under which the binary will be run. This is useful for debugging single binaries (i.e. they need no runfiles) under different contexts.
This PR adds some of that capability to the Bazel plugin by creating a setting for run configurations, and adding it to the Cidr debug process handlers.
Limitations: