Re: svn commit: r1547873 - /subversion/trunk/subversion/libsvn_subr/sqlite.c

2013-12-05 Thread Ben Reser
On 12/5/13 6:13 AM, Philip Martin wrote: > Stefan Sperling writes: > >> I think a hotcopy should be a drop-in replica in most respects, >> so it can be used easily in case the original repository is lost. >> If people want to change access permissions they should do so explicitly. > > I think ho

Re: svn commit: r1547873 - /subversion/trunk/subversion/libsvn_subr/sqlite.c

2013-12-05 Thread Philip Martin
Stefan Sperling writes: > I think a hotcopy should be a drop-in replica in most respects, > so it can be used easily in case the original repository is lost. > If people want to change access permissions they should do so explicitly. I think hotcopy should respect umask. > So copying permission

Re: svn commit: r1547873 - /subversion/trunk/subversion/libsvn_subr/sqlite.c

2013-12-05 Thread Stefan Sperling
On Thu, Dec 05, 2013 at 11:13:42AM +, Philip Martin wrote: > Stefan Sperling writes: > > > On Wed, Dec 04, 2013 at 07:53:40PM +0100, Stefan Sperling wrote: > >> On Wed, Dec 04, 2013 at 06:38:05PM -, phi...@apache.org wrote: > >> > @@ -1314,6 +1314,12 @@ svn_sqlite__hotcopy(const char *src

Re: svn commit: r1547873 - /subversion/trunk/subversion/libsvn_subr/sqlite.c

2013-12-05 Thread Philip Martin
Stefan Sperling writes: > On Wed, Dec 04, 2013 at 07:53:40PM +0100, Stefan Sperling wrote: >> On Wed, Dec 04, 2013 at 06:38:05PM -, phi...@apache.org wrote: >> > @@ -1314,6 +1314,12 @@ svn_sqlite__hotcopy(const char *src_path >> > svn_sqlite__db_t *dst_db; >> > sqlite3_backup *backu

Re: svn commit: r1547873 - /subversion/trunk/subversion/libsvn_subr/sqlite.c

2013-12-04 Thread Stefan Sperling
On Wed, Dec 04, 2013 at 07:53:40PM +0100, Stefan Sperling wrote: > On Wed, Dec 04, 2013 at 06:38:05PM -, phi...@apache.org wrote: > > @@ -1314,6 +1314,12 @@ svn_sqlite__hotcopy(const char *src_path > > svn_sqlite__db_t *dst_db; > > sqlite3_backup *backup; > > int rc1, rc2; > > +

Re: svn commit: r1547873 - /subversion/trunk/subversion/libsvn_subr/sqlite.c

2013-12-04 Thread Stefan Sperling
On Wed, Dec 04, 2013 at 06:38:05PM -, phi...@apache.org wrote: > Author: philip > Date: Wed Dec 4 18:38:04 2013 > New Revision: 1547873 > > URL: http://svn.apache.org/r1547873 > Log: > Another part of issue 3437, rep-cache.db created with wrong permissions. > > * subversion/libsvn_subr/sqlit