Re: Walkthrough for New development on ReviewBoard

2011-01-26 Thread Christian Hammond
Hi Anshul, These are two guides I would love to have time to write -- an introduction to the codebase, and a How To on writing SCMTools. I'll give you a bit of a summary here, though. If you search through the archives, you may find other tidbits of information too. Let's start with the codebase

Re: Hide fields that are not required? (graphics only)

2011-01-26 Thread Christian Hammond
Hi Daniel, If using Review Board 1.5.x, then selecting "None (Graphics only)" for the Repository will hide those fields. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Wed, Jan 26, 2011 at 3:21 PM, Daniel

Walkthrough for New development on ReviewBoard

2011-01-26 Thread Anshul
Hi , I am trying to integrate VSS and an internal SCM of my company with ReviewBoard. I am kind of short on time and unable to find any good walk through on integrating a new SCM with ReviewBoard. Specifically, what all files would i need to modify/add in order to add a new repository from scratc

Hide fields that are not required? (graphics only)

2011-01-26 Thread Daniel
I looked over the documentation but couldn't find an answer... is there any way to hide certain fields that are not required on the 'Create New Review Request' page? 'branch' and even 'diff' may not be relevant when only reviewing graphics. Thanks, Daniel -- Want to help the Review Board projec

Re: API Error 210 (on post-review)

2011-01-26 Thread Christian Hammond
Okay, so that works. Perhaps it's a failure elsewhere. How comfortable are you with Python code? We'd need to add some logging to find out exactly what the failure is. Specifically, in reviewboard/webapi/json.py:repository_info, you want to change: except: return WebAPIResponseError(re

Re: Cannot add repository with cvs ext in 1.5.2 on RHEL5

2011-01-26 Thread praveen kumar
Hi Christian, I am using version 1.5.2 of reviewboard. I do not have any experience with python. However, that shouldn't be a problem. i will look up the internet and learn whatever is required. Please let me know the steps needed to enable more logging in CVS from reviewboard side. Thanks Pravee

Re: API Error 210 (on post-review)

2011-01-26 Thread Ian Battersby
Hi Christian, I get the following information returned (using SilkSVN): Path: tfs2010:8081 URL: http://tfs2010:8081 Repository Root: http://tfs2010:8081 Repository UUID: 6999ed95-b81c-411e-912e-8b8ee48a20fa Revision: 39 Node Kind: directory Last Changed Author: unknown Last Changed Rev: 39 Last C

Re: Cannot add repository with cvs ext in 1.5.2 on RHEL5

2011-01-26 Thread Christian Hammond
Okay, so we'll need to do some extended debugging. What version of Review Board are you using? Do you have any experience with Python? To figure this out in more detail, we'll need to introduce some logging into the CVS support and see how far we're getting and what output we're seeing. Christia

Re: Revision number required for a review.

2011-01-26 Thread Christian Hammond
Hi, Do you mean the revision of the diff being reviewed? When you say review, do you mean a review of a diff, or a review request? I need more clarification on what you need.. So let me see if I have the right understanding. You have a review request with two diffs: Revision 1 and revision 2. Do

Re: API Error 210 (on post-review)

2011-01-26 Thread Christian Hammond
Hi Ian, For Subversion repositories, we attempt to query the information on the remote server for UUID comparisons. It sounds like this is what's failing. I'm guessing your SVNBridge doesn't implement the call we need, or we assume some info is returned that doesn't actually exist there. What hap