Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 2613 by vikas...@gmail.com: Incorrect stripping in CVS repositories
http://code.google.com/p/reviewboard/issues/detail?id=2613

* NOTE: Do not post confidential information in this bug report. *
*       If you need immediate support, please contact            *
*       reviewbo...@googlegroups.com                             *

What version are you running?
1.6.1

What's the URL of the page containing the problem?
Internal page. Not accessible on the web.

What steps will reproduce the problem?
1. Add a CVS repository
2. Upload a diff containing a file that ends with csv
3.

What is the expected output? What do you see instead?
Expected the review request is created, however error generated:
cvs.lin checkout: cannot find module `filename.cs' - ignored

What operating system are you using? What browser?
Linux, Firefox

Please provide any additional information below.
I believe the problem is because of the following line in scmtools/cvs.py
        # Strip off the ",v" we sometimes get for CVS paths.
        if filename.endswith(",v"):
            filename = filename.rstrip(',v')
According to the python documentation, rstrip will strip all characters provided (so instead of ,v being stripped, v,v is)
Changing the above line to
 filename = filename[:-2]
will solve this issue. But I'm not sure if it will have any other side-effects.

Thanks,
Vikas


--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.

Reply via email to