|
From: | Qu Wenruo |
Subject: | Re: [Qemu-devel] [PATCH] gitignore: Ignore vim per-project config and clang complete |
Date: | Mon, 5 Jun 2017 08:45:15 +0800 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 |
At 06/02/2017 10:57 PM, Eric Blake wrote:
On 06/01/2017 11:08 PM, Qu Wenruo wrote:Since we have already ignored ctags, ignoring .clangcomplete for vim-clang_completion is quite reasonable for me. (Not to mention vim-clang_completion provides better and more accurate completion and definition search) Also, qemu coding style is using 4 space as indent other than 8-space tab, it also makes sense to ignore per-project vim config file.We already have a per-project emacs config file (.dir-locals.el); I think it would be a better idea if you were to propose adding an appropriate .vim* file for inclusion in the project that makes vim work out-of-the-box with correct settings, rather than masking the file as unaddable and requiring all vim-using contributors to recreate the same settings. (But as I am not personally a vim user, I can neither provide such a sample file myself, nor review one if you post it).
The problem is there is no out-of-box per-project vimrc behavior for vanilla vim.
There are several ways to do it, like using exrc, which will search for ".vimrc" or ".exrc" in current directory. Although this method will cause annoying prompt every time vim is started up. If this method is preferred, I can change it to .vimrc other than using .vim*.
Or manually call "silent so .vimrc", which is also my preferred method.In case of using "so" method, users can specify the filename as they like. So I use ".vim*".
Although, we have the 3rd method, to match current working path and then apply the per-project settings. This method don't need any modification to qemu, but if one is working on several projects, this could be quite messy.
Thanks, Qu
Signed-off-by: Qu Wenruo <address@hidden> --- .gitignore | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.gitignore b/.gitignore index 55a001e3b8..c7c6eec0b6 100644 --- a/.gitignore +++ b/.gitignore @@ -113,6 +113,8 @@ cscope.* tags TAGS +.vim* +.clang_completeThe idea of adding .clang_complete to the exclusions makes sense to me.
[Prev in Thread] | Current Thread | [Next in Thread] |