Bug 1336 - svn revert fails to operate with files whose names contains "@" character
Summary: svn revert fails to operate with files whose names contains "@" character
Status: RESOLVED WONTFIX
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 2
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Shlomi Fish
QA Contact:
URL:
Whiteboard:
Keywords: UPSTREAM
Depends on:
Blocks:
 
Reported: 2011-05-19 02:51 CEST by Thierry Vignaud
Modified: 2012-05-26 18:38 CEST (History)
5 users (show)

See Also:
Source RPM: subversion
CVE:
Status comment:


Attachments

Description Thierry Vignaud 2011-05-19 02:51:59 CEST
svn revert fails to operate with files whose names contains "@" character:

$ svn di|diffstat 
 sr@Latn.po     |   55 +++++++++++++----------------------------------
 uz@cyrillic.po |   66 +++++++++++++++++++++++++--------------------------------
 2 files changed, 46 insertions(+), 75 deletions(-)
$ svn revert *
'sr' omis
'uz' omis
$ svn revert sr@Latn.po 
'sr' omis
Comment 1 Marja Van Waes 2011-10-13 19:49:11 CEST
ping?

CC: (none) => marja11

Comment 2 Thierry Vignaud 2011-10-13 20:27:13 CEST
still valid
Comment 3 Marja Van Waes 2011-11-11 17:20:13 CET
(In reply to comment #2)
> still valid

no maintainer, so cc'ing last 4 from changelog

CC: (none) => balcaen.john, dmorganec, fundawang, oliver.bgr

Comment 4 Marja Van Waes 2012-01-03 07:58:05 CET
there is a maintainer now, assigning :)

@ Shlomi

Please set status to ASSIGNED if you think this bug was assigned correctly. If for work flow reasons you can't do that, then please put OK on the whiteboard instead.

Assignee: bugsquad => shlomif

Comment 5 Shlomi Fish 2012-01-03 09:16:51 CET
I've tested it and it happens with "svn add" too. This seems to be a problem with the upstream "svn" client too, because our subversion package is very lightly patched. I think the problem is that Subversion uses "@" for peg-revisions.

A workaround is to add another "@foo" at the end. (see the shell transcript below). I'll ask on #svn on Freenode if there's a better way and if it's a known problem:

shlomif@lap:~/tmp/trunk/repos/trunk$ echo FOOBAR > 'hello@bar'
shlomif@lap:~/tmp/trunk/repos/trunk$ ls
hello@bar
shlomif@lap:~/tmp/trunk/repos/trunk$ svn add hello\@bar 
svn: warning: W155010: '/home/shlomif/tmp/trunk/repos/trunk/hello' not found
svn: E200009: Could not add all targets because some targets don't exist
svn: E200009: Illegal target for the requested operation
shlomif@lap:~/tmp/trunk/repos/trunk$ svn add -- hello\@bar 
svn: warning: W155010: '/home/shlomif/tmp/trunk/repos/trunk/hello' not found
svn: E200009: Could not add all targets because some targets don't exist
svn: E200009: Illegal target for the requested operation
shlomif@lap:~/tmp/trunk/repos/trunk$ svn add -- hello@bar 
svn: warning: W155010: '/home/shlomif/tmp/trunk/repos/trunk/hello' not found
svn: E200009: Could not add all targets because some targets don't exist
svn: E200009: Illegal target for the requested operation
shlomif@lap:~/tmp/trunk/repos/trunk$ svn add -- hello@bar ^C
shlomif@lap:~/tmp/trunk/repos/trunk$ svn add hello\@bar 
svn: warning: W155010: '/home/shlomif/tmp/trunk/repos/trunk/hello' not found
svn: E200009: Could not add all targets because some targets don't exist
svn: E200009: Illegal target for the requested operation
shlomif@lap:~/tmp/trunk/repos/trunk$ ls
hello@bar
shlomif@lap:~/tmp/trunk/repos/trunk$ svn add hello\@\@bar 
svn: warning: W155010: '/home/shlomif/tmp/trunk/repos/trunk/hello@' not found
svn: E200009: Could not add all targets because some targets don't exist
svn: E200009: Illegal target for the requested operation
shlomif@lap:~/tmp/trunk/repos/trunk$ svn add hello\\\@bar 
svn: warning: W155010: '/home/shlomif/tmp/trunk/repos/trunk/hello\' not found
svn: E200009: Could not add all targets because some targets don't exist
svn: E200009: Illegal target for the requested operation
shlomif@lap:~/tmp/trunk/repos/trunk$ ls
hello@bar
shlomif@lap:~/tmp/trunk/repos/trunk$ svn st
A       .
?       hello@bar
shlomif@lap:~/tmp/trunk/repos/trunk$ ls
hello@bar
shlomif@lap:~/tmp/trunk/repos/trunk$ ls -l
total 4
-rw-r--r-- 1 shlomif shlomif 7 Jan  3 09:56 hello@bar
shlomif@lap:~/tmp/trunk/repos/trunk$ ls
hello@bar
shlomif@lap:~/tmp/trunk/repos/trunk$ ls -l
total 4
-rw-r--r-- 1 shlomif shlomif 7 Jan  3 09:56 hello@bar
shlomif@lap:~/tmp/trunk/repos/trunk$ svn add hello\@\@bar 
svn: warning: W155010: '/home/shlomif/tmp/trunk/repos/trunk/hello@' not found
svn: E200009: Could not add all targets because some targets don't exist
svn: E200009: Illegal target for the requested operation
shlomif@lap:~/tmp/trunk/repos/trunk$ svn add hello\@bar@foo 
A         hello@bar
shlomif@lap:~/tmp/trunk/repos/trunk$ svn st
A       .
A       hello@bar
shlomif@lap:~/tmp/trunk/repos/trunk$ svn ci
Adding         .
Adding         hello@bar
Transmitting file data .
Committed revision 1.
shlomif@lap:~/tmp/trunk/repos/trunk$ ls
hello@bar
shlomif@lap:~/tmp/trunk/repos/trunk$ ls
hello@bar
shlomif@lap:~/tmp/trunk/repos/trunk$ ls -l
total 4
-rw-r--r-- 1 shlomif shlomif 7 Jan  3 09:56 hello@bar
shlomif@lap:~/tmp/trunk/repos/trunk$ echo WONDERFUL >> hello\@bar 
shlomif@lap:~/tmp/trunk/repos/trunk$ svn di
Index: hello@bar
===================================================================
--- hello@bar   (revision 1)
+++ hello@bar   (working copy)
@@ -1 +1,2 @@
 FOOBAR
+WONDERFUL
shlomif@lap:~/tmp/trunk/repos/trunk$ svn st
M       hello@bar
shlomif@lap:~/tmp/trunk/repos/trunk$ svn revert hello\@bar
Skipped 'hello'
shlomif@lap:~/tmp/trunk/repos/trunk$ svn st
M       hello@bar
shlomif@lap:~/tmp/trunk/repos/trunk$ svn revert hello\@bar\@foo
Reverted 'hello@bar'
shlomif@lap:~/tmp/trunk/repos/trunk$ svn st
shlomif@lap:~/tmp/trunk/repos/trunk$

Status: NEW => ASSIGNED

Comment 6 Marja Van Waes 2012-05-26 13:03:13 CEST
Hi,

This bug was filed against cauldron, but we do not have cauldron at the moment.

Please report whether this bug is still valid for Mageia 2.

Thanks :)

Cheers,
marja

Keywords: (none) => NEEDINFO

Comment 7 Shlomi Fish 2012-05-26 18:38:46 CEST
I think I'll close this bug as UPSTREAM and there's a not-too-hard workaround.

Keywords: NEEDINFO => UPSTREAM
Status: ASSIGNED => RESOLVED
Version: Cauldron => 2
Resolution: (none) => WONTFIX


Note You need to log in before you can comment on or make changes to this bug.