Re: svn commit: r1547866 - /subversion/trunk/subversion/libsvn_fs_fs/rep-cache.c

Philip Martin <philip.martin_at_wandisco.com> writes:

> Stefan Sperling <stsp_at_elego.de> writes:
>
>> Oops, I pasted an old version of my diff. Here's the final one,
>> mostly with adjusted comments.
>>
>> Overall, I like your trick with creating an empty file better.
>>
>> What do you think about making 'svnadmin create' create rep-cache.db?
>
> My new code makes rep-cache.db have umask permissions but I see now that
> is probably not be what we want. When commit creates new revision and
> revprop files it propogates the permissions from the r0 revision file.
> I think that when we create the rep-cache we should do the same.

It's rHEAD rather than r0 but the aim is to make all revision files have
the same permissions so there should be no difference which is copied.
However revision files are not writeable so simple copying won't work
for the rep-cache.

-- 
Philip Martin | Subversion Committer
WANdisco // *Non-Stop Data*