[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: Florin Avram <florin_at_sync.ro>
Date: Mon, 08 Apr 2013 08:51:30 +0300

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).

Regards,
Florin

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?
Received on 2013-04-08 07:52:06 CEST

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.