Re: [PATCH v2] Reject checkouts to existing directory

2016-10-28 Thread Daniel Shahaf
Branko Čibej wrote on Fri, Oct 28, 2016 at 14:22:31 +0200: > On 28.10.2016 09:02, Stefan Sperling wrote: > > On Fri, Oct 28, 2016 at 02:49:04AM +0200, b...@qqmail.nl wrote: > >> In the summary I don’t see the support for restartability of a broken > >> checkout. See the documentation of svn checko

Re: [PATCH v2] Reject checkouts to existing directory

2016-10-28 Thread Johan Corveleyn
On Fri, Oct 28, 2016 at 10:34 AM, Patrick Steinhardt wrote: > On Fri, Oct 28, 2016 at 12:34:44AM +0200, Stefan wrote: >> On 10/27/2016 21:45, Stefan Sperling wrote: >> > On Tue, Oct 18, 2016 at 02:22:25PM +0200, Patrick Steinhardt wrote: >> >> Hi, >> >> >> >> finally got around to update my patch

Re: [PATCH v2] Reject checkouts to existing directory

2016-10-28 Thread Branko Čibej
On 28.10.2016 09:02, Stefan Sperling wrote: > On Fri, Oct 28, 2016 at 02:49:04AM +0200, b...@qqmail.nl wrote: >> In the summary I don’t see the support for restartability of a broken >> checkout. See the documentation of svn checkout in the svn-book. >> (http://svnbook.red-bean.com/nightly/en/svn

Re: [PATCH v2] Reject checkouts to existing directory

2016-10-28 Thread Patrick Steinhardt
On Fri, Oct 28, 2016 at 02:49:04AM +0200, b...@qqmail.nl wrote: > Just checking the log message: This changes 90% of our testsuite to use ‘svn > co --force’, by changing the standard checkout function in actions.py that is > used by virtual all tests. > > If that is really necessary I would call

Re: [PATCH v2] Reject checkouts to existing directory

2016-10-28 Thread Patrick Steinhardt
On Fri, Oct 28, 2016 at 12:34:44AM +0200, Stefan wrote: > On 10/27/2016 21:45, Stefan Sperling wrote: > > On Tue, Oct 18, 2016 at 02:22:25PM +0200, Patrick Steinhardt wrote: > >> Hi, > >> > >> finally got around to update my patch regarding checkouts to > >> existing directories. The semantics have

Re: [PATCH v2] Reject checkouts to existing directory

2016-10-28 Thread Stefan Sperling
On Fri, Oct 28, 2016 at 02:49:04AM +0200, b...@qqmail.nl wrote: > In the summary I don’t see the support for restartability of a broken > checkout. See the documentation of svn checkout in the svn-book. > (http://svnbook.red-bean.com/nightly/en/svn.ref.svn.c.checkout.html) Yes, we'll need to mak

RE: [PATCH v2] Reject checkouts to existing directory

2016-10-27 Thread bert
nderdag 27 oktober 2016 21:45 To: Patrick Steinhardt Cc: Subversion; Stefan Sperling; Ivan Zhakov; Stefan Subject: Re: [PATCH v2] Reject checkouts to existing directory On Tue, Oct 18, 2016 at 02:22:25PM +0200, Patrick Steinhardt wrote: > Hi, > > finally got around to update my patch reg

Re: [PATCH v2] Reject checkouts to existing directory

2016-10-27 Thread Stefan
On 10/27/2016 21:45, Stefan Sperling wrote: > On Tue, Oct 18, 2016 at 02:22:25PM +0200, Patrick Steinhardt wrote: >> Hi, >> >> finally got around to update my patch regarding checkouts to >> existing directories. The semantics have been changed to accept >> checkouts iff >> >> - the target director

Re: [PATCH v2] Reject checkouts to existing directory

2016-10-27 Thread Stefan Sperling
On Tue, Oct 18, 2016 at 02:22:25PM +0200, Patrick Steinhardt wrote: > Hi, > > finally got around to update my patch regarding checkouts to > existing directories. The semantics have been changed to accept > checkouts iff > > - the target directory does not exist > - the target directory is empty

[PATCH v2] Reject checkouts to existing directory

2016-10-18 Thread Patrick Steinhardt
Hi, finally got around to update my patch regarding checkouts to existing directories. The semantics have been changed to accept checkouts iff - the target directory does not exist - the target directory is empty - the repository to check out is empty - the --force flag is given This should trea