Make working with cmake *alot* more convenient #14
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
Fixes problems where the user "cd"s out of the directory in which he called CMake() command, which would make the build dir invalid. Signed-off-by: Squareys <[email protected]>
In some circumstances the user may have already configured a cmake project with a certain generator or other variables, which would be overwritten with the config values set with the g:cmake_* variables. Instead the user should be required to do a clean build if this behaviour is requested. Signed-off-by: Squareys <[email protected]>
Otherwise they will be displayed and then disappear. This way you can re- read the message using :messages Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
Fixed a typo, added doc for recently merged branches and added loads for acknoledgements for people who I found had commits on this project. Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
If found and we are in the build/ directory itself, finddir() will return the full path and not "". Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
Saves an addition string concatenation which reads nicer. Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
See vim documentation. Just have the user use :CMakeClean explicitly, some projects have alot of configuration which should never get deleted by accident when using :CMake. Signed-off-by: Squareys <[email protected]>
Including Vader test and vim documentation Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
Useful when working with multple projects with different build dirs. Also prevents re-searching for the build_dir if already found. Allows specifying a autocommand when switching buffers. Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
Signed-off-by: Squareys <[email protected]>
Squareys
changed the title
|
I almost forgot: In case you merge this, you should create the project on travis/AppVeyor (very straight-forward since the config is already there) and then change the badges in README.md to show the build status of your project instead of my fork ;) |
|
Hi, |
jachjoe
pushed a commit
to jachjoe/vim-cmake
that referenced
this pull request
May 2, 2023
Make working with cmake *alot* more convenient
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi everybody!
First of all: Thank you for creating this plugin! It served me well the last half a year. During that time I noticed some things that would be nice to have, some bugs and some minor annoyances.
The last three days I finally got around working on this plugin a bit and as a result here are my changes. High-Level summary of what I did:
For further information please refer to the commit history and commit messages, everything necessary should be explained there in detail.
I realize this grew to more commits than the project had in total, so I can fully understand if you are not happy with merging this. It would be amazing, though, if you could at least add a reference to my fork in the readme, if you decide not to accept my changes!
Thank you very much, kind regards,
Squareys.
PS: This PR is not compatible with #13