On 2012/03/23 11:33, Marc Espie wrote:
> On Tue, Mar 20, 2012 at 11:42:23AM +0100, Mark Kettenis wrote:
> > > Date: Tue, 20 Mar 2012 10:07:35 +0000
> > > From: Stuart Henderson <s...@spacehopper.org>
> > >
> > > core dumps on OpenBSD are not named 'core' anyway so ignoring them
> > > is pointless and gets in the way.
> > >
> > > OK?
> >
> > Why should cvs on OpenBSD behave different from cvs on other platforms?
>
> Because it already does.
>
> We have already adapted it to current practices, such as ignoring vim
> .swp files.
>
> Do you want this to be reversed as well ?
>
> > People should not name files or directories "core". It leads to
> > accidents like sysadmins accidentally removing stuff when trying to
> > free up disk space by removing all core files.
>
> Nobody is free from idiots. Fortunately, most idiots don't run OpenBSD.
> And even then, idiots won't remove core directories, since rm no longer
> works on directories...
>
>
> Seriously, I really think sthen@'s change should go in.
>
> 1/ It reflects how our system works.
> 2/ Who uses cvs on other platforms anyways ?
> 3/ This is no place for a "holier than thou" dissertation. If people want
> to name files OR DIRECTORIES core, so what ? It's THEIR problem.
> 4/ The current setup of cvs already causes problems, as we REGULARLY have
> software that's organized as core & plugins. I see absolutely NO ISSUE in
> having directories named core.
>
An addition to ignore .git files, some of us use git locally to track
changes pre-commit and, just like the existing ignore entries for
RCS/SCCS/CVS control files, there is no reason to import these to CVS
without an explicit ignore.
Index: src/ignore.c
===================================================================
RCS file: /cvs/src/gnu/usr.bin/cvs/src/ignore.c,v
retrieving revision 1.16
diff -u -p -r1.16 ignore.c
--- src/ignore.c 4 Mar 2012 04:05:15 -0000 1.16
+++ src/ignore.c 26 Mar 2012 09:11:41 -0000
@@ -33,8 +33,8 @@ static int ign_size; /* This many slot
static int ign_hold = -1; /* Index where first "temporary" item
* is held */
-const char *ign_default = ". .. core RCSLOG tags TAGS RCS SCCS .make.state\
- .*.swp\
+const char *ign_default = ". .. RCSLOG tags TAGS RCS SCCS .make.state\
+ .*.swp *.core .git\
.nse_depinfo #* .#* cvslog.* ,* CVS CVS.adm .del-* *.a *.olb *.o *.obj\
*.so *.Z *~ *.old *.elc *.ln *.bak *.BAK *.orig *.rej *.exe _$* *$ *.depend";
