Reviewboard 1.7 -> Reviewboard 2.0.x update

2014-12-18 Thread Daniel Laird
All, We were running Reviewboard 1.7.25 and people used a number of methods to create review requests: svn diff - manually upload post-review rbt I have just migrated us to Reviewboard 2.0.11 and seem to have hit an issue. Some of our reviews no longer display it suggests that a file cant be fo

Re: Reviewboard 1.7 -> Reviewboard 2.0.x update

2014-12-18 Thread Daniel Laird
To follow up the error message we get from the diff viewer is: The file '/svn/Common_PythonTests/http://SVN_SERVER_NAME/svn/Common_PythonTests/path/to/file/base.py' (r1838) could not be found in the repository: '/svn/Common_PythonTests/!svn/rvr/1838/http:/SVN_SERVER/svn/Common_PythonTests/path/

Re: Reviewboard 1.7 -> Reviewboard 2.0.x update

2014-12-18 Thread Christian Hammond
Hi Dan, Would you be able to provide some of the data from that diff (specifically, the 'Index:' line, '---', and '+++' lines in the diff)? A lot of work was done on Subversion since 1.7.x, so something may have regressed. I don't recall being able to handle file paths containing 'http:' in it, t

Re: Reviewboard 1.7 -> Reviewboard 2.0.x update

2014-12-19 Thread Daniel Laird
Christian Many thanks for getting back to me. I think I know why it is breaking but don't know how to fix it. The diff looks as I would expect (with all files being something like this): Index: libs/A/Source/File.cpp === --- libs/A/S

Re: Reviewboard 1.7 -> Reviewboard 2.0.x update

2014-12-19 Thread Christian Hammond
eviewboard 1.7 -> Reviewboard 2.0.x update > Christian > > Many thanks for getting back to me. > I think I know why it is breaking but don't know how to fix it. > > The diff looks as I would expect (with all files being something lik

Re: Reviewboard 1.7 -> Reviewboard 2.0.x update

2014-12-22 Thread Daniel Laird
.com >> > Date: December 19, 2014 at 3:29:19 AM > To: revie...@googlegroups.com >> > Cc: chi...@chipx86.com >> > Subject: Re: Reviewboard 1.7 -> Reviewboard 2.0.x update > > > Christian > > > > Many thanks for getting back to me