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