Skip to content
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

update launch.json for VSCode #8

Merged
merged 13 commits into from
Aug 3, 2021
Merged

update launch.json for VSCode #8

merged 13 commits into from
Aug 3, 2021

Conversation

riebl
Copy link
Collaborator

@riebl riebl commented Jul 26, 2021

As a follow-up task of #7, I have rebased the feature branch and amended the update mechanics. I think this feature is now ready to hit our master branch. I am able to debug with both, GDB and CodeLLDB, on my Linux machine.

You have to explicitly enable the VSCode integration by setting the global VSCODE_ADDON property to ON. The vscode_addon_option command is an easy way for CMake projects to control this property via a cache variable. It is thus easy to disable this feature if someone does not want our code to modify his launch.json.

@HpLightcorner Could you please check if it is (still) working on Windows?

@HpLightcorner
Copy link
Collaborator

@riebl - I will test the implementation and add my thoughts before the final merge.

@HpLightcorner
Copy link
Collaborator

@riebl - sorry for the late response, I was fully concentrating on my simulation. I finally tested all the changes and they work perfectly - also, creating a CMake target is a brilliant solution. I think we can merge.

@riebl riebl merged commit ead9012 into master Aug 3, 2021
@riebl riebl deleted the feature/update_launch_json branch August 3, 2021 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants