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?
