Re: RBTools 0.7.2: posting a review using a range of perforce change lists no longer works

2015-05-05 Thread Christian Hammond
Hi Steve, This looks like a bug with the commit message guessing feature and Perforce revision ranges. Can you try running the same command with '-g no' ? Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com

RBTools 0.7.2: posting a review using a range of perforce change lists no longer works

2015-05-05 Thread Steve
I've been doing a fair amount of testing of RBTools in preparation for moving to RB 2.0 and I've come across a few important issues that are preventing us from moving to RBTools 0.7. The 2 most significant ones appear to be regressions as my tests pass in 0.6.3 but fail in 0.7.2. The first

Re: RBTools 0.7.2: posting a review using a range of perforce change lists no longer works

2015-05-05 Thread Steve
Hi Chris, The good news is, your diagnosis is correct. Use of '-g no' got past the error. The bad news is, the diff posted is not correct. But I found that the diff posted with 0.6.3 is the same incorrect diff (it's incomplete and appears to be a single change instead of a range), so I'm

Re: how to provide comment disposition in review board

2015-05-05 Thread Christian Hammond
I'm sorry, I don't understand what you're asking. Can you go into more detail on what you're looking for? Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com On Tue, May 5, 2015 at 12:12 AM, deepak sahoo

Re: RBTools 0.7.2: posting a review using a range of perforce change lists no longer works

2015-05-05 Thread Christian Hammond
Hi Steve, Glad it hear it (with the exception of the diff not being correct). I'd appreciate a bug report on it. I won't be able to get to this right away, so having all these details will help for later. Thanks! Christian -- Christian Hammond - chip...@chipx86.com Review Board -

Re: Facing two different errors while installing new reviewboard and migrating postgresql database from 1.5.5 to new version.

2015-05-05 Thread Sri Ram Kannan
Hi Chris, I have solved the second issue. It is due to the Import Error. The login needs LDAP module and when I installed it, I was able to login successfully. Thanks for your reply. Regards, Sri. On Tuesday, May 5, 2015 at 4:59:00 AM UTC-4, Christian Hammond wrote: Hi, The first error

Re: Exclude in .reviewboardrc

2015-05-05 Thread Jeff Buzzell
Barret, This works for me. Thank you for your help. EXCLUDE_PATTERNS = [ r'*\AssemblyInfo.cs', r'*\Controllers\*', r'*\Test\*' ] On Monday, May 4, 2015 at 3:53:41 PM UTC-6, Barret Rennie wrote: Hi there Jeff, When using exclude patterns in your .reviewboardrc file, it takes a

Re: Issue after ReviewBoard upgrade

2015-05-05 Thread Papaswede
Yeah, I've tried various combinations of permissions and always restarting apache after I make changes. I'm a bit stuck here as the error messages do not make any sense to me. On Thursday, April 30, 2015 at 9:47:00 PM UTC-4, David Trowbridge wrote: Have you tried restarting the web server

Issue 3862 in reviewboard: Loading Patch hangs due to french local

2015-05-05 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 3862 by msu...@aurea.com: Loading Patch hangs due to french local https://code.google.com/p/reviewboard/issues/detail?id=3862 *** READ THIS BEFORE POSTING! *** *** You must complete this form in its entirety, or your bug

View diff and collapse left or right side

2015-05-05 Thread Jeff Buzzell
Is there a way to collapse the left or right side of the diff view? I often need to search for keywords in the code that appear in both sides of the diff. Browser find works horizontally and bounces back and forth between the left and right diff sides when keywords appear in the same line of

Re: View diff and collapse left or right side

2015-05-05 Thread Christian Hammond
Hi Jeff, There's no way to do this, I'm afraid. Something to consider for a future version, but it'll probably be a bit low on the priority list for now, relative to other projects we're working on. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org

Re: Issue after ReviewBoard upgrade

2015-05-05 Thread Christian Hammond
If the path is referencing an incorrect directory, then it sounds like you may be using a database originally installed in a different path. You can verify this by typing: $ rb-site manage /path/to/site list-siteconfig That will show you all your configured site settings, some of which will

Re: Facing two different errors while installing new reviewboard and migrating postgresql database from 1.5.5 to new version.

2015-05-05 Thread Christian Hammond
Hi, The first error isn't an error. It's just part of the upgrade process. It looks like it completed successfully. As for the second one, can you see what's in your reviewboard.log or apache error log file? Christian -- Christian Hammond - chip...@chipx86.com Review Board -

Facing two different errors while installing new reviewboard and migrating postgresql database from 1.5.5 to new version.

2015-05-05 Thread Sri Ram Kannan
Hi, My first error is: * Installing the site... Building site directories ... OK Building site configuration files ... OK Creating database ... Creating tables ... Creating table extensions_registeredextension Creating table accounts_localsiteprofile Creating table attachments_fileattachment

Re: Review Board Installation: Bad Request (400)

2015-05-05 Thread Jiangcheng Liu
This is really tricky! I spent almost 30 hours to fix this problem( Bad Request (400) ) *And the real solution is: /var/www/reviews/conf/**settings_local.py ALLOWED_HOSTS = ['localhost'] to ALLOWED_HOSTS = ['*'].* On Wednesday, June 4, 2014 at 6:04:52 AM UTC-7, Kuldeep singh wrote:

how to provide comment disposition in review board

2015-05-05 Thread deepak sahoo
I am using review board for my code review tool.But how to put comment disposition like and about the status of the review whether it is open,or closed or conflicted.. -- Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/ Want us to host Review Board for

Re: Issue 3855 in reviewboard: rbt post -r ID changenum does not update the change list number in the review request

2015-05-05 Thread reviewboard
Comment #1 on issue 3855 by seide.al...@gmail.com: rbt post -r ID changenum does not update the change list number in the review request https://code.google.com/p/reviewboard/issues/detail?id=3855 It looks like the code that should be updating the commit_id is on line 512 of post.py, but

Issue 3861 in reviewboard: Support setting blocks field in review requests the same way as depends_on field is set.

2015-05-05 Thread reviewboard
Status: New Owner: Labels: Type-Enhancement Priority-Medium New issue 3861 by csipak.a...@ardinsys.eu: Support setting blocks field in review requests the same way as depends_on field is set. https://code.google.com/p/reviewboard/issues/detail?id=3861 What version are you running?