Issue 2545 in reviewboard: Review Board does not understand patches that span directories included via svn:externals

2012-03-21 Thread reviewboard

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

New issue 2545 by bill...@gmail.com: Review Board does not understand  
patches that span directories included via svn:externals

http://code.google.com/p/reviewboard/issues/detail?id=2545

* 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.4.1

What's the URL of the page containing the problem?

/reviewboard/r/new/

What steps will reproduce the problem?
1. Create a directory in subversion, /a/
2. Create a directory in subversion, /b/
3. In /a/, create a subversion property ^/b b, so that /b/ is included as  
/a/b/

4. Create and 'svn add' two files, /a/a.txt, /a/b/b.txt.
5. Create a patch from /a/, containint those two files.
6. Submit the patch using New Review Request

What is the expected output? What do you see instead?

The expected output is to create a review as if /a/b/ had been a normal  
directory.  Instead, an error message is displayed: The  
file 'https://svnserver/svn/a/b/b.txt' (r12345) could not be found in the  
repository


What operating system are you using? What browser?

n/a

Please provide any additional information below.

n/a

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



Re: Issue 2545 in reviewboard: Review Board does not understand patches that span directories included via svn:externals

2012-03-21 Thread reviewboard


Comment #1 on issue 2545 by bill...@gmail.com: Review Board does not  
understand patches that span directories included via svn:externals

http://code.google.com/p/reviewboard/issues/detail?id=2545

Sorry, step 3 should read

3. In /a/, create a subversion svn:externals property ^/b b, so that  
/b/ is included as /a/b/


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