Skip to content

Feature | Adds command to open the cmake cache file#309

Merged
Civitasv merged 1 commit intoCivitasv:masterfrom
BenLeadbetter:feature/open-cmake-cache
Apr 22, 2025
Merged

Feature | Adds command to open the cmake cache file#309
Civitasv merged 1 commit intoCivitasv:masterfrom
BenLeadbetter:feature/open-cmake-cache

Conversation

@BenLeadbetter
Copy link
Copy Markdown
Contributor

The Feature

Adds a new command to open the cmake cache file. CLion has this functionality which I always found super helpful.

Considerations

In this initial implementation the file is opened simply with a vim edit <cache file> command. Being a vim classicist this suites my workflow, but I can see that other commands (for example CMakeOpenRunner) open a new split window. Would this be preferable?

@lceWolf
Copy link
Copy Markdown
Collaborator

lceWolf commented Apr 16, 2025

Thanks for your contribution!
Personally, I never used the cache file that much - except for skimming trough it.
What is your way of working with that file?
Opening it in the current buffer might be a legitimate approach for frequent edits.
If you only want to have a glance at it or make a quick one-line edit, opening it in a floating window (like CMakeSettings does) might also be worth a consideration.

I think I wouldn't make it open in a split like the runner/executer. They are kinda terminal-like instances for an ongoing process.

WDYT @Civitasv?

@BenLeadbetter
Copy link
Copy Markdown
Contributor Author

What is your way of working with that file?

I'm often editing the cache file to switch on and off the various configuration options a project might expose during development. After editing cmake is smart enough to detect the change and regenerate the next time a build is kicked off.

@Civitasv
Copy link
Copy Markdown
Owner

I think it's okay to open it in current buffer.

@Civitasv Civitasv merged commit 1ee065c into Civitasv:master Apr 22, 2025
1 of 2 checks passed
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.

3 participants