-
Notifications
You must be signed in to change notification settings - Fork 450
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
Fix for Github issue 587: duplicate project structure causes error #1169
Conversation
DirectoryNodes use PREFIX/subdir
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.
I didn't fully understand your comment on #587. Is there still a bug that this PR doesn't address? I tried the example you posted in the comment as well as the repo you linked in this PR and didn't see a problem with either of them.
i updated the code in the reproduction repository. project(base2) was supposed to be commented out. Additional explanation:
But i found the issue. |
Thank you so much! |
This PR resolves: #587
Changes
Added project sourceDirectory to class ProjectNode.
_rootDir is now initialized with sourceDirectory as prefix.
Added projectDirectory to tooltip of ProjectNodes
Notes
Minimal project to reproduce issue:
https://github.com/SchweizS/vscode-cmake-tools587