Re: pre-commit review request with svn integration

2014-07-09 Thread neel roy
anyone? specially about #4? On Friday, July 4, 2014 7:37:02 PM UTC+5:30, neel roy wrote: Hi, This is what I want to do. I am sure it's not possible today but can I know a) whether you find it useful enough to put in the next release or b) is there a way for me to write an extension that

Re: Getting an error There was an error displaying this diff while doing diff in review request.

2014-07-09 Thread Nitish Gangal
Hi Christian, Thanks for your reply. As you suggested I tried using RBtools and post command to create a review request like: rbt post --server=http://10.0.14.29 -d But, I got 500 as a response. Detailed error is as follows: Got HTTP error: 500: !DOCTYPE html html head title500 -

Re: Getting an error There was an error displaying this diff while doing diff in review request.

2014-07-09 Thread Nitish Gangal
Hey Christian, I resolved the issue mentioned in my previous comment. It was because I had two repositories added with same mirror path. I rectified it. So, now using rbt post --server=http://10.0.14.29 -d, I am able to successfully create a review request. Still, when I click on 'view diff'

Re: installation problems - debian

2014-07-09 Thread Guy Klose
Thanks, indeed that was exactly the problem. Had to go back and do this (or so it seemed): # wget https://bootsrap.pypa.io/ez_setup.py -O - | python After that, I went through the apt-get install steps (python, python-setuptools, python-dev, etc). Proceeded with the site setup instructions,

Re: Is there a procedure for copying a RB 1.6.16 DB/site to a new server with RB 2.0.2 installed?

2014-07-09 Thread Stephen
When I try this on the command line it turns out I get nothing on stdout and an error on stderr so that seems to be the cause. The error was //depot/path/to/file#revision - protected namespace - access denied. I've contacted my Perforce admin to help with the issue. If I use a different user

Re: 404 Errors with review-board 2.0.2 with lighttpd

2014-07-09 Thread Michael Kopp
On July 7, 2014 at 11:06:38 AM, Michael Kopp (michael.f.k...@googlemail.com) wrote: I've been trying to set up review-board on a lighttpd-server (v1.4.35) under arch linux.  I managed to set it up to display something, but unfortunately all the static data such as css, images and js are not

Re: pre-commit review request with svn integration

2014-07-09 Thread David Trowbridge
Are you wanting it so that Review Board commits the code, or do you want it so that `svn commit` will only succeed if the code has been reviewed? -David On Wed, Jul 9, 2014 at 12:36 AM, neel roy neelsm...@rediffmail.com wrote: anyone? specially about #4? On Friday, July 4, 2014 7:37:02 PM

Re: 404 Errors with review-board 2.0.2 with lighttpd

2014-07-09 Thread Christian Hammond
On July 9, 2014 at 1:50:31 PM, Michael Kopp (michael.f.k...@googlemail.com) wrote: On July 7, 2014 at 11:06:38 AM, Michael Kopp (michael.f.k...@googlemail.com) wrote:  I've been trying to set up review-board on a lighttpd-server (v1.4.35)  under arch linux.  I managed to set it up to display

Re: captcha for registration broken link on RB2.0.2

2014-07-09 Thread MoonWalker
done :-) On Wednesday, July 9, 2014 5:30:26 AM UTC+10, Christian Hammond wrote: Loos like. Can you file a bug? Christian -- Christian Hammond - chri...@beanbaginc.com javascript: Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com On June 24, 2014 at

Re: Authentication failed while adding svn repository

2014-07-09 Thread shravanthi s
I have verified that credentials are correct and i can login using same credentials manually into svn. I also entered required credentials while adding repository info. It still gives the same error. Am not sure what else can go wrong here. Do you need any other information? I dont know what

Re: Issue 3455 in reviewboard: Redundant information related to review of files should be removed

2014-07-09 Thread reviewboard
Comment #3 on issue 3455 by joanna.p...@gmail.com: Redundant information related to review of files should be removed http://code.google.com/p/reviewboard/issues/detail?id=3455 I am afraid that it is not an issue of permissions. Take a look at pictures attached. In some cases src is empty

Issue 3473 in reviewboard: Browsing to /reviews/r/id/diff/rev/fragment/random number causes Internal Server Error

2014-07-09 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 3473 by erik.l...@gmail.com: Browsing to /reviews/r/id/diff/rev/fragment/random number causes Internal Server Error http://code.google.com/p/reviewboard/issues/detail?id=3473 What version are you running? 2.0.2 What's

Re: Issue 3473 in reviewboard: Browsing to /reviews/r/id/diff/rev/fragment/random number causes Internal Server Error

2014-07-09 Thread reviewboard
Comment #1 on issue 3473 by erik.l...@gmail.com: Browsing to /reviews/r/id/diff/rev/fragment/random number causes Internal Server Error http://code.google.com/p/reviewboard/issues/detail?id=3473 Adding simple patch. Attachments: Issue3473.patch 535 bytes -- You received this

Re: Issue 1094 in reviewboard: Copy/paste source code

2014-07-09 Thread reviewboard
Comment #4 on issue 1094 by stephene...@gmail.com: Copy/paste source code http://code.google.com/p/reviewboard/issues/detail?id=1094 I think this would be worth doing, it's a very frequent need. I don't think it's due to a specific browser, most browsers will behave this way. There are

Re: Issue 1094 in reviewboard: Copy/paste source code

2014-07-09 Thread reviewboard
Comment #5 on issue 1094 by chip...@gmail.com: Copy/paste source code http://code.google.com/p/reviewboard/issues/detail?id=1094 You would think it's easy, but we've spent a lot of time on it, and there's no clean way of handling this. While it's possible to dynamically enable/disable

Re: Issue 48 in reviewboard: Add number of lines changed to dashboard view

2014-07-09 Thread reviewboard
Comment #11 on issue 48 by trowb...@gmail.com: Add number of lines changed to dashboard view http://code.google.com/p/reviewboard/issues/detail?id=48 Issue 3472 has been merged into this issue. -- You received this message because this project is configured to send all issue notifications

Re: Issue 3472 in reviewboard: Review size to be able to prioritize which reviews to tackle

2014-07-09 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 48 Comment #1 on issue 3472 by trowb...@gmail.com: Review size to be able to prioritize which reviews to tackle http://code.google.com/p/reviewboard/issues/detail?id=3472 In Review Board 2.0.x, there's a Diff Size column which will show