[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH v2] Reject checkouts to existing directory

From: Stefan <luke1410_at_posteo.de>
Date: Fri, 28 Oct 2016 00:34:44 +0200

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 directory does not exist
>> - the target directory is empty
>> - the repository to check out is empty
>> - the --force flag is given
> I think this makes a lot of sense. I just have not yet had time to
> review and test your patch. I will try to do so soon.
>
> Does anyone reading this list have any concerns about this change?

I'm +1 on the general design/behavior change. Didn't do a code/patch
review, though.

On a minor side note:
While talking last week on IRC to Daniel, he mentioned (on a different
topic) that in general it might be preferable to use a separate explicit
command line options to control the exact behavior over one which
impacts several behaviors at once. Reflecting that onto this case, it
crossed my mind that --allow-non-empty-directory (or --allow-non-empty)
might be preferable over adding that behavior to the --force parameter,
since the --force parameter has (or in the future might have) other
implications in addition to allowing a co into a non-empty directory.

Though in this case, I don't have a strong opinion to go one way or the
other.

Regards,
Stefan

Received on 2016-10-28 00:35:04 CEST

This is an archived mail posted to the Subversion Dev mailing list.