Re: After upgrading to ReviewBoard 2.0.15 the existing reviews do not display the diffs correctly

2015-05-29 Thread Papaswede
Oh ok, thanks for the info! We typically just use the IDE to generate a 
.patch file and then upload it to reviewboard. The projects in our IDE are 
usually set up as one for trunk and one for each branch, so when it 
generates the .patch files it generates them with relative paths using 
/trunk or /branches/branchversion as the root directory, so that's why 
we have the repositories configured that way in ReviewBoard. If we change 
the repo configuration in ReviewBoard around to not target the individual 
branches then we'd have to start hand editing the patch files after 
generating them to make sure the path in there references the full path 
including the branch. It looks like this woud be the case even if using 
RBTools because I don't see a way for RBTools to target a changelist, or am 
I missing something? We might have 4 or more separate changelists at any 
given time in the IDE and those could encompass hundreds of files, so 
targeting a whole changelist for posting the review is pretty crucial for 
us. Is there a way for RBTools to do this? 



On Thursday, May 28, 2015 at 6:32:06 PM UTC-4, Christian Hammond wrote:

 Hi, 

 If svn/Repo1, svn/Repo2, etc. are all independent Subversion repositories, 
 and not subdirectories within a larger repository, then you should have a 
 repository entry in Review Board for each. 

 You shouldn't target trunk, branches, etc., because those are 
 subdirectories. When posting a change for review against a Subversion 
 repository, RBTools will send paths that are relative to the root of the 
 repository (say, /trunk/myproject/README). That then gets accessed relative 
 to the repository path, giving you something like 
 https://vcs.ourdomain.com/svn/Repo1/trunk/myproject/README, or, if 
 misconfigured, something like 
 https://vcs.ourdomain.com/svn/Repo1/trunk/trunk/myproject/README. 

 To fix things up by hand, you'll need to modify the diffviewer_filediff 
 table entries (or FileDiff in the database editor in the admin UI) and 
 change the paths shown there. The trick is associating the entries with a 
 given review request. For that, you can try something like: 

 $ rb-site manage /path/to/site shell 
  from reviewboard.diffviewer.models import FileDiff 
  print 
 FileDiff.objects.filter(diffset__history__review_request=review request 
 ID).values_list('pk', 'source_file') 

 Where review request ID is replaced with the numeric ID of the review 
 reuqest. That would show you all FileDiff IDs and their paths for a given 
 review request. 

 Christian 

 --   
 Christian Hammond - chri...@beanbaginc.com javascript:   
 Review Board - https://www.reviewboard.org   
 Beanbag, Inc. - https://www.beanbaginc.com 

 -Original Message- 
 From: Papaswede papa...@gmail.com javascript: 
 Reply: revie...@googlegroups.com javascript: revie...@googlegroups.com 
 javascript: 
 Date: May 28, 2015 at 2:32:20 PM 
 To: revie...@googlegroups.com javascript: revie...@googlegroups.com 
 javascript: 
 Subject:  Re: After upgrading to ReviewBoard 2.0.15 the existing reviews 
 do not display the diffs correctly 

  Ok, thank you. 

  If I wanted to go through and fix the paths of the existing reviews, 
 which 
  tables, specifically, would I need to go through and look at? 

  Aslo, this may be a dumb question, but I'm not the most Subversion savvy 
  person in the world. We have our repositories set up like 

  https://vcs.ourdomain.com/svn/ 

  and then we have the Repositories within there, like 

  https://vcs.ourdomain.com/svn/Repo1 
  https://vcs.ourdomain.com/svn/Repo2 

  and then each repository has 

  https://vcs.ourdomain.com/svn/Repo1/trunk 
  https://vcs.ourdomain.com/svn/Repo1/branches 
  https://vcs.ourdomain.com/svn/Repo1/branches/2012.REL 
  https://vcs.ourdomain.com/svn/Repo1/branches/2013.REL 
  ... 

  Are you saying we'd need to point straight to 
  https://vcs.ourdomain.com/svn/Repo1/ in order for Reviewboard to work 
  correctly, or can we target trunk or individual branches, etc.? 

  On Wednesday, May 27, 2015 at 12:28:40 PM UTC-4, Papaswede wrote: 
  
  I upgraded ReviewBoard from 1.6.11 to 2.0.15. After doing this, I get 
  error messages when attempting to view the diffs of the existing 
 reviews. I 
  get messages like: 
  
  The file 
  
 '('/svn/enrollment/!svn/bc/122232/trunk/4X/enrollment/trunk/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java'
  

  path not found, 160013)' could not be found in the repository 
  
  Where that path definitely is not a valid svn path. It should 
  be 
 /svn/enrollment/!svn/bc/122232/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java.
  

  
  So, it's like there's stuff being automatically inserted into the svn 
 path 
  for the file, but I have no idea why as it was working on version 
 1.6.11. 
  
  Any help would be greatly appreciated. 
  

  -- 
  Supercharge your Review Board with Power Pack: 
 

Re: After upgrading to ReviewBoard 2.0.15 the existing reviews do not display the diffs correctly

2015-05-29 Thread Christian Hammond
When manually uploading Subversion diffs, Review Board will prompt you for the 
base path of the diff. This is the path relative to the root of the 
Subversion repository where the diff was generated in. For instance, it may be 
/trunk or /branches/branchversion. Whatever you specify will be prepended 
to the paths in the diff.

RBTools calculates this automatically and sets it for you when posting a change 
for review, but you can always enter it by hand when manually uploading the 
diff through the web UI.

Christian

--  
Christian Hammond - christ...@beanbaginc.com  
Review Board - https://www.reviewboard.org  
Beanbag, Inc. - https://www.beanbaginc.com

-Original Message-
From: Papaswede papasw...@gmail.com
Reply: Papaswede papasw...@gmail.com
Date: May 29, 2015 at 7:32:26 AM
To: reviewboard@googlegroups.com reviewboard@googlegroups.com
Cc: christ...@beanbaginc.com christ...@beanbaginc.com
Subject:  Re: After upgrading to ReviewBoard 2.0.15 the existing reviews do not 
display the diffs correctly

 Oh ok, thanks for the info! We typically just use the IDE to generate a
 .patch file and then upload it to reviewboard. The projects in our IDE are
 usually set up as one for trunk and one for each branch, so when it
 generates the .patch files it generates them with relative paths using
 /trunk or /branches/branchversion as the root directory, so that's why
 we have the repositories configured that way in ReviewBoard. If we change
 the repo configuration in ReviewBoard around to not target the individual
 branches then we'd have to start hand editing the patch files after
 generating them to make sure the path in there references the full path
 including the branch. It looks like this woud be the case even if using
 RBTools because I don't see a way for RBTools to target a changelist, or am
 I missing something? We might have 4 or more separate changelists at any
 given time in the IDE and those could encompass hundreds of files, so
 targeting a whole changelist for posting the review is pretty crucial for
 us. Is there a way for RBTools to do this?
  
  
  
 On Thursday, May 28, 2015 at 6:32:06 PM UTC-4, Christian Hammond wrote:

 Hi,

 If svn/Repo1, svn/Repo2, etc. are all independent Subversion repositories,
 and not subdirectories within a larger repository, then you should have a
 repository entry in Review Board for each.

 You shouldn't target trunk, branches, etc., because those are
 subdirectories. When posting a change for review against a Subversion
 repository, RBTools will send paths that are relative to the root of the
 repository (say, /trunk/myproject/README). That then gets accessed relative
 to the repository path, giving you something like
 https://vcs.ourdomain.com/svn/Repo1/trunk/myproject/README, or, if
 misconfigured, something like
 https://vcs.ourdomain.com/svn/Repo1/trunk/trunk/myproject/README.

 To fix things up by hand, you'll need to modify the diffviewer_filediff
 table entries (or FileDiff in the database editor in the admin UI) and
 change the paths shown there. The trick is associating the entries with a
 given review request. For that, you can try something like:

 $ rb-site manage /path/to/site shell
  from reviewboard.diffviewer.models import FileDiff
  print
 FileDiff.objects.filter(diffset__history__review_request=review request
 ID).values_list('pk', 'source_file')

 Where review request ID is replaced with the numeric ID of the review
 reuqest. That would show you all FileDiff IDs and their paths for a given
 review request.

 Christian

 --
 Christian Hammond - chri...@beanbaginc.com javascript:
 Review Board - https://www.reviewboard.org
 Beanbag, Inc. - https://www.beanbaginc.com

 -Original Message-
 From: Papaswede papa...@gmail.com javascript:
 Reply: revie...@googlegroups.com javascript: revie...@googlegroups.com
 javascript:
 Date: May 28, 2015 at 2:32:20 PM
 To: revie...@googlegroups.com javascript: revie...@googlegroups.com
 javascript:
 Subject: Re: After upgrading to ReviewBoard 2.0.15 the existing reviews
 do not display the diffs correctly

  Ok, thank you.
 
  If I wanted to go through and fix the paths of the existing reviews,
 which
  tables, specifically, would I need to go through and look at?
 
  Aslo, this may be a dumb question, but I'm not the most Subversion savvy
  person in the world. We have our repositories set up like
 
  https://vcs.ourdomain.com/svn/
 
  and then we have the Repositories within there, like
 
  https://vcs.ourdomain.com/svn/Repo1
  https://vcs.ourdomain.com/svn/Repo2
 
  and then each repository has
 
  https://vcs.ourdomain.com/svn/Repo1/trunk
  https://vcs.ourdomain.com/svn/Repo1/branches
  https://vcs.ourdomain.com/svn/Repo1/branches/2012.REL
  https://vcs.ourdomain.com/svn/Repo1/branches/2013.REL
  ...
 
  Are you saying we'd need to point straight to
  https://vcs.ourdomain.com/svn/Repo1/ in order for Reviewboard to work
  correctly, or can we target trunk or individual branches, etc.?
 
  On 

Re: After upgrading to ReviewBoard 2.0.15 the existing reviews do not display the diffs correctly

2015-05-28 Thread Papaswede
Ok, thank you.

If I wanted to go through and fix the paths of the existing reviews, which 
tables, specifically, would I need to go through and look at?


On Wednesday, May 27, 2015 at 12:28:40 PM UTC-4, Papaswede wrote:

 I upgraded ReviewBoard from 1.6.11 to 2.0.15. After doing this, I get 
 error messages when attempting to view the diffs of the existing reviews. I 
 get messages like:

 The file 
 '('/svn/enrollment/!svn/bc/122232/trunk/4X/enrollment/trunk/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java'
  
 path not found, 160013)' could not be found in the repository

 Where that path definitely is not a valid svn path. It should 
 be 
 /svn/enrollment/!svn/bc/122232/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java.

 So, it's like there's stuff being automatically inserted into the svn path 
 for the file, but I have no idea why as it was working on version 1.6.11.

 Any help would be greatly appreciated.


-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: After upgrading to ReviewBoard 2.0.15 the existing reviews do not display the diffs correctly

2015-05-28 Thread Christian Hammond
Hi,

If svn/Repo1, svn/Repo2, etc. are all independent Subversion repositories, and 
not subdirectories within a larger repository, then you should have a 
repository entry in Review Board for each.

You shouldn't target trunk, branches, etc., because those are subdirectories. 
When posting a change for review against a Subversion repository, RBTools will 
send paths that are relative to the root of the repository (say, 
/trunk/myproject/README). That then gets accessed relative to the repository 
path, giving you something like 
https://vcs.ourdomain.com/svn/Repo1/trunk/myproject/README, or, if 
misconfigured, something like 
https://vcs.ourdomain.com/svn/Repo1/trunk/trunk/myproject/README.

To fix things up by hand, you'll need to modify the diffviewer_filediff table 
entries (or FileDiff in the database editor in the admin UI) and change the 
paths shown there. The trick is associating the entries with a given review 
request. For that, you can try something like:

    $ rb-site manage /path/to/site shell
     from reviewboard.diffviewer.models import FileDiff
     print FileDiff.objects.filter(diffset__history__review_request=review 
request ID).values_list('pk', 'source_file')

Where review request ID is replaced with the numeric ID of the review 
reuqest. That would show you all FileDiff IDs and their paths for a given 
review request.

Christian

--  
Christian Hammond - christ...@beanbaginc.com  
Review Board - https://www.reviewboard.org  
Beanbag, Inc. - https://www.beanbaginc.com

-Original Message-
From: Papaswede papasw...@gmail.com
Reply: reviewboard@googlegroups.com reviewboard@googlegroups.com
Date: May 28, 2015 at 2:32:20 PM
To: reviewboard@googlegroups.com reviewboard@googlegroups.com
Subject:  Re: After upgrading to ReviewBoard 2.0.15 the existing reviews do not 
display the diffs correctly

 Ok, thank you.
  
 If I wanted to go through and fix the paths of the existing reviews, which
 tables, specifically, would I need to go through and look at?
  
 Aslo, this may be a dumb question, but I'm not the most Subversion savvy
 person in the world. We have our repositories set up like
  
 https://vcs.ourdomain.com/svn/
  
 and then we have the Repositories within there, like
  
 https://vcs.ourdomain.com/svn/Repo1
 https://vcs.ourdomain.com/svn/Repo2
  
 and then each repository has
  
 https://vcs.ourdomain.com/svn/Repo1/trunk
 https://vcs.ourdomain.com/svn/Repo1/branches
 https://vcs.ourdomain.com/svn/Repo1/branches/2012.REL
 https://vcs.ourdomain.com/svn/Repo1/branches/2013.REL
 ...
  
 Are you saying we'd need to point straight to
 https://vcs.ourdomain.com/svn/Repo1/ in order for Reviewboard to work
 correctly, or can we target trunk or individual branches, etc.?
  
 On Wednesday, May 27, 2015 at 12:28:40 PM UTC-4, Papaswede wrote:

 I upgraded ReviewBoard from 1.6.11 to 2.0.15. After doing this, I get
 error messages when attempting to view the diffs of the existing reviews. I
 get messages like:

 The file
 '('/svn/enrollment/!svn/bc/122232/trunk/4X/enrollment/trunk/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java'
 path not found, 160013)' could not be found in the repository

 Where that path definitely is not a valid svn path. It should
 be 
 /svn/enrollment/!svn/bc/122232/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java.

 So, it's like there's stuff being automatically inserted into the svn path
 for the file, but I have no idea why as it was working on version 1.6.11.

 Any help would be greatly appreciated.

  
 --
 Supercharge your Review Board with Power Pack: 
 https://www.reviewboard.org/powerpack/
 Want us to host Review Board for you? Check out RBCommons: 
 https://rbcommons.com/
 Happy user? Let us know! https://www.reviewboard.org/users/
 ---
 You received this message because you are subscribed to the Google Groups 
 reviewboard
 group.
 To unsubscribe from this group and stop receiving emails from it, send an 
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/d/optout.
  

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: After upgrading to ReviewBoard 2.0.15 the existing reviews do not display the diffs correctly

2015-05-28 Thread Christian Hammond
Hi,

Review Board was never intended to work with Subversion repository paths
that point to a subdirectory. Repositories should always point to the root
of the repository, because RBTools and the New Review Request page are
going to try to generate paths that are relative to the root of the
repository. That's how those ended up with the extra trunk in the path a
couple of times.

It looks like you have a mix of changes on your server that either have the
right path or the wrong path. I don't know why it would have worked before
but not now, but we've fixed a large number of Subversion issues since
1.6.x, and rewrote the Subversion backend, so perhaps there's a behavioral
change somewhere. I'd have to know more about how those particular changes
were posted to really help diagnose that.

The right thing to do here is to switch the repository URL to not point to
a subdirectory. However, that will break existing review requests. So what
I'd recommend is archiving the current repository entry (and keeping the
path as-is), and then adding a new repository entry pointing to the root of
the server, and give it the same name. RBTools and the New Review Request
page will do the correct thing, and you won't have this sort of issue going
forward.

That won't fix existing review requests, though. For that, either the
database entries would need to be updated to point to the correct path, or
a patch to Review Board would need to be written that tries to remove that
duplicate information when normalizing the path, for misconfigured
repositories.

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
Beanbag, Inc. - http://www.beanbaginc.com

On Thu, May 28, 2015 at 6:41 AM, Papaswede papasw...@gmail.com wrote:

 Oh, and I just noticed that a few of the existing reviews still do work,
 but about 90% of them do not. I was clicking through some at random and
 found that some do show up correctly. I'm not sure what's wrong with the
 ones that aren't showing up.

 On Wednesday, May 27, 2015 at 12:28:40 PM UTC-4, Papaswede wrote:

 I upgraded ReviewBoard from 1.6.11 to 2.0.15. After doing this, I get
 error messages when attempting to view the diffs of the existing reviews. I
 get messages like:

 The file
 '('/svn/enrollment/!svn/bc/122232/trunk/4X/enrollment/trunk/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java'
 path not found, 160013)' could not be found in the repository

 Where that path definitely is not a valid svn path. It should
 be 
 /svn/enrollment/!svn/bc/122232/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java.

 So, it's like there's stuff being automatically inserted into the svn
 path for the file, but I have no idea why as it was working on version
 1.6.11.

 Any help would be greatly appreciated.

  --
 Supercharge your Review Board with Power Pack:
 https://www.reviewboard.org/powerpack/
 Want us to host Review Board for you? Check out RBCommons:
 https://rbcommons.com/
 Happy user? Let us know! https://www.reviewboard.org/users/
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/d/optout.


-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: After upgrading to ReviewBoard 2.0.15 the existing reviews do not display the diffs correctly

2015-05-28 Thread Papaswede
Ok, thank you.

If I wanted to go through and fix the paths of the existing reviews, which 
tables, specifically, would I need to go through and look at?

Aslo, this may be a dumb question, but I'm not the most Subversion savvy 
person in the world. We have our repositories set up like

https://vcs.ourdomain.com/svn/

and then we have the Repositories within there, like

https://vcs.ourdomain.com/svn/Repo1
https://vcs.ourdomain.com/svn/Repo2

and then each repository has

https://vcs.ourdomain.com/svn/Repo1/trunk
https://vcs.ourdomain.com/svn/Repo1/branches
https://vcs.ourdomain.com/svn/Repo1/branches/2012.REL
https://vcs.ourdomain.com/svn/Repo1/branches/2013.REL
...

Are you saying we'd need to point straight to 
https://vcs.ourdomain.com/svn/Repo1/ in order for Reviewboard to work 
correctly, or can we target trunk or individual branches, etc.?

On Wednesday, May 27, 2015 at 12:28:40 PM UTC-4, Papaswede wrote:

 I upgraded ReviewBoard from 1.6.11 to 2.0.15. After doing this, I get 
 error messages when attempting to view the diffs of the existing reviews. I 
 get messages like:

 The file 
 '('/svn/enrollment/!svn/bc/122232/trunk/4X/enrollment/trunk/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java'
  
 path not found, 160013)' could not be found in the repository

 Where that path definitely is not a valid svn path. It should 
 be 
 /svn/enrollment/!svn/bc/122232/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java.

 So, it's like there's stuff being automatically inserted into the svn path 
 for the file, but I have no idea why as it was working on version 1.6.11.

 Any help would be greatly appreciated.


-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: After upgrading to ReviewBoard 2.0.15 the existing reviews do not display the diffs correctly

2015-05-28 Thread Papaswede
The URL for this one is https://vcs.ourdomain.com/svn/enrollment/trunk/4X. 
That isn't the root per se, but we have multiple different repositories 
under our svn so we point it directly at the folder we want as the root, 
and then when we create the reviews we use / as the base path for the 
review. The URL is the same as it was on Reviewboard 1.6.11 and it was 
working there. Also, if I create a brand new review against this repo it 
works fine. It's just when I try to navigate to reviews that existed before 
the upgrade that the issue occurs.

On Wednesday, May 27, 2015 at 12:28:40 PM UTC-4, Papaswede wrote:

 I upgraded ReviewBoard from 1.6.11 to 2.0.15. After doing this, I get 
 error messages when attempting to view the diffs of the existing reviews. I 
 get messages like:

 The file 
 '('/svn/enrollment/!svn/bc/122232/trunk/4X/enrollment/trunk/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java'
  
 path not found, 160013)' could not be found in the repository

 Where that path definitely is not a valid svn path. It should 
 be 
 /svn/enrollment/!svn/bc/122232/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java.

 So, it's like there's stuff being automatically inserted into the svn path 
 for the file, but I have no idea why as it was working on version 1.6.11.

 Any help would be greatly appreciated.


-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: After upgrading to ReviewBoard 2.0.15 the existing reviews do not display the diffs correctly

2015-05-28 Thread Papaswede
Here's a screenshot of the repository configuration:

https://lh3.googleusercontent.com/-XjVj3czBb1g/VWcSv6TMhPI/AL0/jCi0XFHd1kk/s1600/Change_repository___Administration___Review_Board.png


On Wednesday, May 27, 2015 at 12:28:40 PM UTC-4, Papaswede wrote:

 I upgraded ReviewBoard from 1.6.11 to 2.0.15. After doing this, I get 
 error messages when attempting to view the diffs of the existing reviews. I 
 get messages like:

 The file 
 '('/svn/enrollment/!svn/bc/122232/trunk/4X/enrollment/trunk/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java'
  
 path not found, 160013)' could not be found in the repository

 Where that path definitely is not a valid svn path. It should 
 be 
 /svn/enrollment/!svn/bc/122232/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java.

 So, it's like there's stuff being automatically inserted into the svn path 
 for the file, but I have no idea why as it was working on version 1.6.11.

 Any help would be greatly appreciated.


-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: After upgrading to ReviewBoard 2.0.15 the existing reviews do not display the diffs correctly

2015-05-28 Thread Papaswede
Oh, and I just noticed that a few of the existing reviews still do work, 
but about 90% of them do not. I was clicking through some at random and 
found that some do show up correctly. I'm not sure what's wrong with the 
ones that aren't showing up.

On Wednesday, May 27, 2015 at 12:28:40 PM UTC-4, Papaswede wrote:

 I upgraded ReviewBoard from 1.6.11 to 2.0.15. After doing this, I get 
 error messages when attempting to view the diffs of the existing reviews. I 
 get messages like:

 The file 
 '('/svn/enrollment/!svn/bc/122232/trunk/4X/enrollment/trunk/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java'
  
 path not found, 160013)' could not be found in the repository

 Where that path definitely is not a valid svn path. It should 
 be 
 /svn/enrollment/!svn/bc/122232/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java.

 So, it's like there's stuff being automatically inserted into the svn path 
 for the file, but I have no idea why as it was working on version 1.6.11.

 Any help would be greatly appreciated.


-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


After upgrading to ReviewBoard 2.0.15 the existing reviews do not display the diffs correctly

2015-05-27 Thread Papaswede
I upgraded ReviewBoard from 1.6.11 to 2.0.15. After doing this, I get error 
messages when attempting to view the diffs of the existing reviews. I get 
messages like:

The file 
'('/svn/enrollment/!svn/bc/122232/trunk/4X/enrollment/trunk/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java'
 
path not found, 160013)' could not be found in the repository

Where that path definitely is not a valid svn path. It should 
be 
/svn/enrollment/!svn/bc/122232/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java.

So, it's like there's stuff being automatically inserted into the svn path 
for the file, but I have no idea why as it was working on version 1.6.11.

Any help would be greatly appreciated.

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: After upgrading to ReviewBoard 2.0.15 the existing reviews do not display the diffs correctly

2015-05-27 Thread Christian Hammond
Hi,

Can you show us the repository configuration? It looks like you may not be
pointing to the root of the SVN repository.

Christian


On Wednesday, May 27, 2015, Papaswede papasw...@gmail.com wrote:

 I upgraded ReviewBoard from 1.6.11 to 2.0.15. After doing this, I get
 error messages when attempting to view the diffs of the existing reviews. I
 get messages like:

 The file
 '('/svn/enrollment/!svn/bc/122232/trunk/4X/enrollment/trunk/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java'
 path not found, 160013)' could not be found in the repository

 Where that path definitely is not a valid svn path. It should
 be 
 /svn/enrollment/!svn/bc/122232/trunk/4X/webapps/common/src/main/java/bf/web/common/form/AddEmployeeForm.java.

 So, it's like there's stuff being automatically inserted into the svn path
 for the file, but I have no idea why as it was working on version 1.6.11.

 Any help would be greatly appreciated.

 --
 Supercharge your Review Board with Power Pack:
 https://www.reviewboard.org/powerpack/
 Want us to host Review Board for you? Check out RBCommons:
 https://rbcommons.com/
 Happy user? Let us know! https://www.reviewboard.org/users/
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com
 javascript:_e(%7B%7D,'cvml','reviewboard%2bunsubscr...@googlegroups.com');
 .
 For more options, visit https://groups.google.com/d/optout.



-- 
-- 
Christian Hammond - christ...@beanbaginc.com
Review Board - http://www.reviewboard.org
Beanbag, Inc. - http://www.beanbaginc.com

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.