Re: Add comments via API

2014-02-20 Thread Karel Novotny
Hi, thanks ... so I did a small exercise for myself, I'll try this ReviewBot K Dne čtvrtek, 20. února 2014 1:19:54 UTC+1 Javins, Walt napsal(a): Hi Karel, I’m aware of at least one other community project that does automated static code analysis:

Re: Review Board with support for older Rails version(for code review) and support for Team Foundation Server(TFS) as the SCM tool?

2014-02-20 Thread Mohnish Jadwani
Thank you David. On Monday, February 17, 2014 5:06:34 PM UTC, Mohnish Jadwani wrote: Dear All, I'm working on a Rails 2.1 app which uses Ruby 1.8.7. Will review board help in code review of an older version of Rails ? I'm sorry this might sound like a basic question, but I don't know

Re: The current directory does not contain a checkout... (problem posting from perforce)

2014-02-20 Thread Darryl McCutcheon
I'm seeing a couple issues with the perforce integration as well. The one above with the case mismatch, as well as the --p4-client parameter is not respected in RBTools 0.5.7. An integration with eclipse and reviewboard (https://code.google.com/p/p4reviewboard/) breaks because this parameter

Re: Unicode + ldap login traceback

2014-02-20 Thread David Trowbridge
Walt, Thanks. We'll get this fixed for the next 1.7.x point release. -David On Wed, Feb 19, 2014 at 3:10 PM, Javins, Walt walt.jav...@isilon.comwrote: Thanks David! I’ve filed the bug here: http://code.google.com/p/reviewboard/issues/detail?id=3256 Let me know what else I can do

Re: The current directory does not contain a checkout... (problem posting from perforce)

2014-02-20 Thread David Trowbridge
Darryl, The --p4-* options have a bug where settings in a .p4config file still had higher priority than the command-line options. I'll make sure that the fix for that bug is in the next rbtools point release. Thanks, -David On Thu, Feb 20, 2014 at 6:16 AM, Darryl McCutcheon

Re: Pre-Commit Workflow

2014-02-20 Thread Charles Yates
On Wednesday, 19 February 2014 16:36:21 UTC+1, Allen wrote: Really good question! I also encounter the same problem. Developers dont want to wait until the previous review passed to work on the next issue. One solution I think is, once a review passed, apply that review`s patch file to

Re: Pre-Commit Workflow

2014-02-20 Thread Charles Yates
On Wednesday, 19 February 2014 21:15:31 UTC+1, David Trowbridge wrote: The typical way that other dvcs users do this is to have local branches for each piece of work, and post from each of those. I don't know enough about bzr's workflow to know if that would be appropriate (though I recall

Re: Pre-Commit Workflow

2014-02-20 Thread Christian Hammond
On Wed, Feb 19, 2014 at 7:36 AM, Allen zhangsan8...@gmail.com wrote: I have no idea why the review board changes the paths from local to remote server, at least it should keep the local working copy`s path. Here`s a example of what those paths look like: ---

Review Board 2.0 beta 3 is released!

2014-02-20 Thread Christian Hammond
Hey everyone, We've just released 2.0 beta 3, which is a pretty significant release, and the last beta in the 2.0 cycle. We've done a lot of polish and made a number of improvements in this beta, and we'd love to have you give it a try. The news announcement, and links to release notes and

Re: Issue 3045 in reviewboard: when updating new diff on existing reviews on reviewboard server in clearcase source file revision is not LATEST

2014-02-20 Thread reviewboard
Comment #17 on issue 3045 by saar.net...@gmail.com: when updating new diff on existing reviews on reviewboard server in clearcase source file revision is not LATEST http://code.google.com/p/reviewboard/issues/detail?id=3045 Thanks Allot :-) Adding the --publish did the trick :-) -- You

Issue 3257 in reviewboard: LDAP Authentication Error with Email Attribute

2014-02-20 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 3257 by frikoda...@gmail.com: LDAP Authentication Error with Email Attribute http://code.google.com/p/reviewboard/issues/detail?id=3257 What version are you running? 1.7.21 What's the URL of the page containing the

Re: Issue 3257 in reviewboard: LDAP Authentication Error with Email Attribute

2014-02-20 Thread reviewboard
Comment #1 on issue 3257 by frikoda...@gmail.com: LDAP Authentication Error with Email Attribute http://code.google.com/p/reviewboard/issues/detail?id=3257 I'm using BitNami pre-packaged version of ReviewBoard 1.7.21 and python-ldap-2.4.13.win32-py2.7 package for LDAP support. -- You

Re: Issue 3256 in reviewboard: unable to log in (http 500) when using ldap auth backend + unicode password

2014-02-20 Thread reviewboard
Updates: Status: Fixed Comment #2 on issue 3256 by trowb...@gmail.com: unable to log in (http 500) when using ldap auth backend + unicode password http://code.google.com/p/reviewboard/issues/detail?id=3256 Fixed in release-1.7.x (55b7c92). Thanks! -- You received this message

Re: Issue 3257 in reviewboard: LDAP Authentication Error with Email Attribute

2014-02-20 Thread reviewboard
Updates: Status: Fixed Comment #4 on issue 3257 by trowb...@gmail.com: LDAP Authentication Error with Email Attribute http://code.google.com/p/reviewboard/issues/detail?id=3257 Fixed in release-1.7.x (f32c957). Thanks! -- You received this message because this project is configured

Re: Issue 3252 in reviewboard: http://downloads.reviewboard.org/nightlies/ does not exist

2014-02-20 Thread reviewboard
Updates: Status: Fixed Comment #3 on issue 3252 by trowb...@gmail.com: http://downloads.reviewboard.org/nightlies/ does not exist http://code.google.com/p/reviewboard/issues/detail?id=3252 Fixed in release-2.0.x (5430221). Thanks! -- You received this message because this project is