sergey-visual-studio/dpack

Can't assign bookmarks when using a CMake project

TheNicker opened this issue · 6 comments

image

Windows 10 21H2m
Visual studio 17.1.3

Test project: Test.zip

  1. Open the test project as CMake project,
  2. Navigate to Test.cpp and try to toggle a bookmark by pressing Ctrl+Shift+[Number]
  3. BUG: a bookmark isn't being assigned

Will investigate, thanks. Sorry about the delay in response. Somehow I stopped getting Github notifications.

I'll need help figuring out what makes CMake menu item appear there. It's missing in my environment unfortunately.

You also say "open test project". I don't see any project or solution file in .zip attached. Please clarify. Thanks.

Add CMake tools for windows as depicted below:

  1. Open tools and features
    image

  2. Add C++ CMake tools
    image

The issue happens when using a CMake project file
In the root folder inside the zip there's a file called CMakeLists.txt. That's the project/solution file.

File->Open->Cmake-> Then choose the file CMakeLists.txt and the project will open.

In Visual studio 2019 with DPack there's no issue, Though DPackRX with VS2022 has this issue, just checked with the current latest version VS2022 17.4.0

Take a look at issue #43 where I posted a beta version. Give it a try to see if that helps before I start getting into this issue.

@sergey-visual-studio , thanks for your reply.
Tried with DPackRx-4.5.2.1.vsix.zip
Unfortunately it didn't solve the issue.

Reproduced it here.

In Visual studio 2019 with DPack there's no issue,

I don't see how that'd be the case tbhwy. For a few reasons: 1 - it doesn't work the same exact way for me on VS 2019, 2 - both versions of DPack bookmarks feature have identical command availability logic, and 3 - both versions require solution to be opened for bookmark commands to be available. That last point is what prevents it from enabling the bookmarks cause as far as VS goes, it reports no open solution. There is quite a bit of DPack logic that relies on open solution. I'll check if relaxing that requirement is feasible or not. I suspect not.