Skip to content

Small improvements to CMake folders#1857

Open
dgelessus wants to merge 2 commits intoH-uru:masterfrom
dgelessus:improve_cmake_folders
Open

Small improvements to CMake folders#1857
dgelessus wants to merge 2 commits intoH-uru:masterfrom
dgelessus:improve_cmake_folders

Conversation

@dgelessus
Copy link
Copy Markdown
Contributor

Fixes the folder for the Scripts target to avoid this suboptimal display in CLion:

CLion's Run Configurations menu, showing a folder with an empty name containing a target named Scripts

and moves the many, many targets related to clang-tidy into a folder named tidy so that they don't fill up the CMakePredefinedTargets default folder.

With my build configuration, the folder CMakePredefinedTargets now still contains the targets check, gmock, gmock_main, gtest, gtest_main, and tools. Should I move any of those into different folders too (Tests and Tools presumably)? None of these targets come from CMake, so it's a bit surprising that they're in CMakePredefinedTargets.

(For context: the CMake folder structure was originally added in #1612.)

If the FOLDER property is set to an empty string, CLion 2025.3 takes
that literally and puts the target into a folder with a blank name.
Unsetting the FOLDER property puts the target at top level as expected.
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.

1 participant