[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#37215: [PATCH] vc-cvs-ignore writes absolute filenames and duplicate
From: |
Eli Zaretskii |
Subject: |
bug#37215: [PATCH] vc-cvs-ignore writes absolute filenames and duplicate strings |
Date: |
Fri, 14 Feb 2020 10:33:48 +0200 |
> Cc: larsi@gnus.org, 37215@debbugs.gnu.org
> From: Wolfgang Scherer <Wolfgang.Scherer@gmx.de>
> Date: Fri, 14 Feb 2020 02:24:12 +0100
>
> >> Your assumption, that `file` is always a simple basename is wrong.
> > Yes, but when does it make sense to have FILE not absolute and not
> > just a basename (i.e. with leading directories)? Do we have such use
> > cases?
> vc-dir-ignore with patch from #37240
OK, but then please document this use case and how DIRECTORY is used
in this function. The various -ignore functions in vc.el and in
backends assign different semantics to their DIRECTORY argument, and I
think these (largely undocumented) differences are a source of some
confusion and bugs in this area.
> > Because if that happens, the file's name will be added to
> > .cvsignore not in DIRECTORY but in one of its subdirectories. Would
> > that be surprising?
> Not for anybody familiar with CVS.
This should be documented, IMO. The existing documentation of
.cvsignore is minimal, and doesn't mention several important aspects.
For example, the fact that only basenames are allowed is only hinted
upon, and there's no information whatsoever AFAICT whether characters
special to wildcards can be escaped. So I think we should provide
this minimal information either in doc strings or at least in comments
in the code.
Thanks.
- bug#37215: [PATCH] vc-cvs-ignore writes absolute filenames and duplicate strings, Eli Zaretskii, 2020/02/13
- bug#37215: [PATCH] vc-cvs-ignore writes absolute filenames and duplicate strings, Wolfgang Scherer, 2020/02/13
- bug#37215: [PATCH] vc-cvs-ignore writes absolute filenames and duplicate strings,
Eli Zaretskii <=
- bug#37215: [PATCH] vc-cvs-ignore writes absolute filenames and duplicate strings, Wolfgang Scherer, 2020/02/15
- bug#37215: [PATCH] vc-cvs-ignore writes absolute filenames and duplicate strings, Wolfgang Scherer, 2020/02/19
- bug#37215: [PATCH] vc-cvs-ignore writes absolute filenames and duplicate strings, Eli Zaretskii, 2020/02/21
- bug#37215: [PATCH] vc-cvs-ignore writes absolute filenames and duplicate strings, Dmitry Gutov, 2020/02/21
- bug#37215: [PATCH] vc-cvs-ignore writes absolute filenames and duplicate strings, Wolfgang Scherer, 2020/02/21
- bug#37215: [PATCH] vc-cvs-ignore writes absolute filenames and duplicate strings, Dmitry Gutov, 2020/02/21