On Mon, Mar 18, 2024 at 13:14:54 +0100, Michal Prívozník wrote:
On 3/18/24 12:45, Denis V. Lunev wrote:
> On 3/18/24 11:42, Michal Prívozník wrote:
>> On 3/17/24 16:00, Denis V. Lunev wrote:
>>> Signed-off-by: Denis V. Lunev <den(a)openvz.org>
>>> ---
>>> .gitignore | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/.gitignore b/.gitignore
>>> index 4695391342..44a9b446bd 100644
>>> --- a/.gitignore
>>> +++ b/.gitignore
>>> @@ -20,6 +20,7 @@ __pycache__/
>>> /build/
>>> /ci/scratch/
>>> tags
>>> +cscope.*
>>> # clangd related ignores
>>> .clangd
>> Apparently, at some point in time this was here, but it was removed:
>>
>>
https://gitlab.com/libvirt/libvirt/-/commit/8a63add283c310952b7df161b4b41...
>>
>> Michal
>>
> This is quite strange for me:
>
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/....
>
https://gitlab.com/qemu-project/qemu/-/blob/master/.gitignore?ref_type=heads
>
> I do not see any obvious reason how big and extensive
> list of files in .gitignore could hurt us while it
> is obviously convenient.
Yeah, it feels a bit selective. I mean, we allow vim swap files to be in
.gitignore, we allow tags file to be there too (which strictly speaking
is a tooling helper file), but not cscope?
The reasoning to keep 'tags' ignored is because we ship .ctags, but
generally tooling files should be ignored by "personal" .gitignores as
we can't cover all of it especially if we don't configure it.
I guess the reasoning for editor temp files is to prevent mistakes as
those are basically to be present for everyone trying to edit the
project.