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

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.

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.

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

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

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

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

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

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

Re: Issue 1959 in reviewboard: Looking for a list of DIFF file settings - using Mercurial 1.7.3

2011-01-26 Thread reviewboard
Updates: Labels: -Type-Defect Type-Support Comment #1 on issue 1959 by chip...@gmail.com: Looking for a list of DIFF file settings - using Mercurial 1.7.3 http://code.google.com/p/reviewboard/issues/detail?id=1959 We expect a unified diff in a format that our tools can parse. A

Re: Issue 1885 in reviewboard: Permissions to restrict viewing of all commits and closing review requests

2011-01-26 Thread reviewboard
Comment #1 on issue 1885 by ruben.st...@mevis.fraunhofer.de: Permissions to restrict viewing of all commits and closing review requests http://code.google.com/p/reviewboard/issues/detail?id=1885 I would appreciate that feature too. At our institute there are projects whose code is

Re: Issue 1632 in reviewboard: Alphabetial sorting of review requests should not be case sensitive

2011-01-26 Thread reviewboard
Comment #8 on issue 1632 by jira.s...@gmail.com: Alphabetial sorting of review requests should not be case sensitive http://code.google.com/p/reviewboard/issues/detail?id=1632 This seems to be database related (the order_by() function in django can order by a list of fields from a table,

Re: Issue 1953 in reviewboard: Addressing comments on View Diff page is not usable

2011-01-26 Thread reviewboard
Comment #2 on issue 1953 by jeffl...@gmail.com: Addressing comments on View Diff page is not usable http://code.google.com/p/reviewboard/issues/detail?id=1953 This issue is definitely what I'm experiencing as well, but it's not very succinct. I'll try to explain in a different way. Look

Re: Issue 1953 in reviewboard: Addressing comments on View Diff page is not usable

2011-01-26 Thread reviewboard
Comment #3 on issue 1953 by jeffl...@gmail.com: Addressing comments on View Diff page is not usable http://code.google.com/p/reviewboard/issues/detail?id=1953 This issue is definitely what I'm experiencing as well, but it's not very succinct. I'll try to explain in a different way. Look

Re: Issue 1953 in reviewboard: Addressing comments on View Diff page is not usable

2011-01-26 Thread reviewboard
Comment #4 on issue 1953 by jeffl...@gmail.com: Addressing comments on View Diff page is not usable http://code.google.com/p/reviewboard/issues/detail?id=1953 This issue is definitely what I'm experiencing as well, but it's not very succinct. I'll try to explain in a different way. Look

Issue 1961 in reviewboard: Git SSH repository: Cannot spawn rbssh

2011-01-26 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1961 by anthony@gmail.com: Git SSH repository: Cannot spawn rbssh http://code.google.com/p/reviewboard/issues/detail?id=1961 *NOTE: Do not post confidential information in this bug report.* What version are you

Re: Issue 1961 in reviewboard: Git SSH repository: Cannot spawn rbssh

2011-01-26 Thread reviewboard
Updates: Status: NeedInfo Comment #1 on issue 1961 by chip...@gmail.com: Git SSH repository: Cannot spawn rbssh http://code.google.com/p/reviewboard/issues/detail?id=1961 This looks to me like there are a couple problems. First, your Apache configuration likely was never updated to

Re: Issue 1961 in reviewboard: Git SSH repository: Cannot spawn rbssh

2011-01-26 Thread reviewboard
Comment #2 on issue 1961 by anthony@gmail.com: Git SSH repository: Cannot spawn rbssh http://code.google.com/p/reviewboard/issues/detail?id=1961 Thanks for writing back quickly! I see now that another user had overridden the system path to point at a different python scripts directory,

Re: Issue 1961 in reviewboard: Git SSH repository: Cannot spawn rbssh

2011-01-26 Thread reviewboard
Comment #3 on issue 1961 by anthony@gmail.com: Git SSH repository: Cannot spawn rbssh http://code.google.com/p/reviewboard/issues/detail?id=1961 2011-01-26 17:28:23,963 - ERROR - Git: Failed to find valid repository ssh://xxx.xxx.com/somerepo.git:

Re: Issue 1961 in reviewboard: Git SSH repository: Cannot spawn rbssh

2011-01-26 Thread reviewboard
Comment #4 on issue 1961 by anthony@gmail.com: Git SSH repository: Cannot spawn rbssh http://code.google.com/p/reviewboard/issues/detail?id=1961 P.S. I have pycrypto 2.1 installed for python 2.5 -- You received this message because you are subscribed to the Google Groups

Re: Issue 1961 in reviewboard: Git SSH repository: Cannot spawn rbssh

2011-01-26 Thread reviewboard
Comment #5 on issue 1961 by chip...@gmail.com: Git SSH repository: Cannot spawn rbssh http://code.google.com/p/reviewboard/issues/detail?id=1961 pwd is a Linux-only Python module, as far as I can tell. It looks like Python assumes it can access it, even though it can't. This is outside our

Re: Issue 1961 in reviewboard: Git SSH repository: Cannot spawn rbssh

2011-01-26 Thread reviewboard
Comment #6 on issue 1961 by anthony@gmail.com: Git SSH repository: Cannot spawn rbssh http://code.google.com/p/reviewboard/issues/detail?id=1961 Ah, good idea. I made my username part of the SSH URL now and it is able to skip over the pwd call. It now complains that the authentication

Re: Issue 1961 in reviewboard: Git SSH repository: Cannot spawn rbssh

2011-01-26 Thread reviewboard
Comment #7 on issue 1961 by chip...@gmail.com: Git SSH repository: Cannot spawn rbssh http://code.google.com/p/reviewboard/issues/detail?id=1961 It *should* try both password and public key. Did you try re-generating/uploading a key since making the %HOME% change? If so, it no longer