Re: Initial Move Support in /trunk now

2013-09-23 Thread Julian Foad
Stefan Fuhrmann wrote: > As of r1525442, there is a new svn_fs_move() API supported > by all backends with BDB providing only rudimentary support. Cool!  Thanks for starting the implementation, Stefan. > It turned out that move() should not implicitly issue a delete() > on the source node as this

Re: Initial Move Support in /trunk now

2013-09-23 Thread Ivan Zhakov
On 23 September 2013 13:34, Stefan Fuhrmann wrote: > On Mon, Sep 23, 2013 at 9:49 AM, Ivan Zhakov wrote: >> >> On 23 September 2013 02:51, Stefan Fuhrmann >> wrote: >> > >> > Hi there, >> > >> > As of r1525442, there is a new svn_fs_move() API supported >> > by all backends with BDB providing on

Re: Initial Move Support in /trunk now

2013-09-23 Thread Stefan Fuhrmann
On Mon, Sep 23, 2013 at 9:49 AM, Ivan Zhakov wrote: > On 23 September 2013 02:51, Stefan Fuhrmann > wrote: > > > > Hi there, > > > > As of r1525442, there is a new svn_fs_move() API supported > > by all backends with BDB providing only rudimentary support. > > > > It turned out that move() shoul

Re: Initial Move Support in /trunk now

2013-09-23 Thread Branko Čibej
On 23.09.2013 11:25, Stefan Fuhrmann wrote: > On Mon, Sep 23, 2013 at 9:30 AM, Branko Čibej > wrote: > > On 23.09.2013 00:51, Stefan Fuhrmann wrote: > > Hi there, > > > > As of r1525442, there is a new svn_fs_move() API supported > > by all backends w

Re: Initial Move Support in /trunk now

2013-09-23 Thread Stefan Fuhrmann
On Mon, Sep 23, 2013 at 9:30 AM, Branko Čibej wrote: > On 23.09.2013 00:51, Stefan Fuhrmann wrote: > > Hi there, > > > > As of r1525442, there is a new svn_fs_move() API supported > > by all backends with BDB providing only rudimentary support. > > So if I read the logs correctly, you did nothing

Re: Initial Move Support in /trunk now

2013-09-23 Thread Ivan Zhakov
On 23 September 2013 02:51, Stefan Fuhrmann wrote: > > Hi there, > > As of r1525442, there is a new svn_fs_move() API supported > by all backends with BDB providing only rudimentary support. > > It turned out that move() should not implicitly issue a delete() > on the source node as this creates a

Re: Initial Move Support in /trunk now

2013-09-23 Thread Branko Čibej
On 23.09.2013 00:51, Stefan Fuhrmann wrote: > Hi there, > > As of r1525442, there is a new svn_fs_move() API supported > by all backends with BDB providing only rudimentary support. So if I read the logs correctly, you did nothing to fix the copy-id generation, or to preserve it for moved nodes?