Thorsten Glaser writes ("Re: Bug#858769: "cvs init" creates CVSROOT/history and 
val-tags world-writeable"):
> On further reflection, I removed the chmod() calls, making the
> permission bits of both files depend only on ${CVSUMASK:-002}
> as is customary for other files in the repository (including
> updating the documentation accordingly).

This sounds to me like the correct fix.

> The regular Unix umask is used for files in the working copy
> (the checkout). This is to distinguish between these two, as
> people will normally work with umask 022 and CVSUMASK 002 or
> possibly 007 to facilitate sharing a repository among a Unix
> user group.

Right.

> Of course, this means I’ll have to restart running the insane
> testsuite of this… but hey, progress! In the meanwhile, if you
> wish to inspect the whole patchset (I’ll have to split off the
> parts I can justify getting into stretch despite the freeze):

If I get some time I may look at this but TBH I have a lot of crises
right now (including the alarming original events which prompted me to
file this bug) so don't wait for me !

Thanks,
Ian.

-- 
Ian Jackson <ijack...@chiark.greenend.org.uk>   These opinions are my own.

If I emailed you from an address @fyvzl.net or @evade.org.uk, that is
a private address which bypasses my fierce spamfilter.

Reply via email to