Re: [PATCH 1/1] gitignore: add cscope files to git ignore

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx>
> >>> ---
> >>>   .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/8a63add283c310952b7df161b4b413e817468d01
> >>
> >> Michal
> >>
> > This is quite strange for me:
> > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/.gitignore
> > 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.
_______________________________________________
Devel mailing list -- devel@xxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxx




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux