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

Re: SVN 1.7 - can delete external files

From: Daniel Shahaf <danielsh_at_elego.de>
Date: Mon, 8 Apr 2013 11:01:30 +0300

[CC += dev@, so fullquote; please drop users@ on replies]

Florin Avram wrote on Mon, Apr 08, 2013 at 08:51:30 +0300:
> On 05.04.2013 17:41, Stefan Sperling wrote:
>> On Fri, Apr 05, 2013 at 05:12:10PM +0300, Syncro SVN Client Support wrote:
>>> Hi,
>>>
>>> I found that you can delete external files with SVN 1.7, which
>>> should not be allowed. Below are some test I've done with SVN 1.6
>>> and SVN 1.7 over external files and directories.
>>>
>>> *I.**external file*
>>> *1.**SVN 1.6* - neither "svn delete external.txt" or "svn delete
>>> --keep-local external.txt" work (which is fine, according with the
>>> SVN Book).
>>> svn: E155030: Cannot remove the file external at 'external.txt';
>>> please propedit or propdel the svn:externals description that
>>> created itII. external dir
>>>
>>> *2**. SVN 1.7.8* - "svn delete external.txt" throws same error (OK).
>>> But, "svn delete --keep-local" works, file is removed and the
>>> EXTERNAL status is lost.
>>>> svn delete --keep-local external.txt
>>> D external.txt
>>>
>>>> svn status external.txt
>>> D external.txt
>>>
>>>> svn status . (on parent directory)
>>> D external.txt
>>>
>>>> svn commit -m "test" external.txt
>>> Deleting external.txt
>>> Committed revision 3128.
>>> *
>>>
>>> **II**.**external dir*
>>> SVN 1.6.17 allows deleting external directories, but SVN 1.7 denies
>>> deletion of external directories (an error like "cannot delete the
>>> root of a working copy").
>>>
>>>
>>> In the end, I assume "svn delete" should not work for an external
>>> item, ignoring the SVN version, the item type (file or dir) and any
>>> option for delete (like --force or --keep-local), no?
>>>
>>> Regards,
>>> Florin
>> Yes, I agree.
>>
>> The trunk code had the same issue, but with slightly different output.
>> Fixed in http://svn.apache.org/r1464992
>>
>> Do you think this is an important enough issue to warrant a backport to 1.7?
> Hi,
>
> Well, I don't know how often this situation can be seen, so it is up to
> you (SVN developers) to decide what to do (I just signaled the issue).
>

What happens after someone deletes (or deletes-and-commits) a file
external? Would revert+cleanup restore the working copy to a valid
state, or would he need to checkout a fresh wc?

Daniel

> Regards,
> Florin
>
>
Received on 2013-04-08 10:02:12 CEST

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