rbt post ignoring PUBLISH in .reviewboardrc?

2014-02-27 Thread Lee Laborczfalvi
I've just switched over to use rbt instead of post-review, but I'm finding that it's ignoring the PUBLISH=1 directive in the .reviewboardrc file. I can see that my review is sitting on the server with the diff correctly there, the distribution group correctly there, but it is a draft only, not

Re: 2.0 Beta3 bugs

2014-02-27 Thread Allen
Yeah, I will. Thanks On Wednesday, February 26, 2014 6:26:41 PM UTC-5, David Trowbridge wrote: Can you file these on the google code bug tracker? Thanks! -David On Wed, Feb 26, 2014 at 6:55 AM, Allen zhangs...@gmail.com javascript:wrote: I am wondering is this a good place to report

Reviewboard ClearCase configuration

2014-02-27 Thread Khalil Shalish
Hi, I am not clear on how to configure Reviewboard (1.7) with our clearcase setup. Our clearcase setup: We use clearcase dynamic views on linux. The user can have a view or set of views on his client machine. The views are mounted and accessed in the following manner:

Re: upload diff file getting problem

2014-02-27 Thread Christian Hammond
You'll need to generate diffs by passing --full-index to git diff. We recommend though that you use RBTools to post instead, as it'll ensure the diff is in the right format, attach other useful data, and create/update the review request for you. Christian On Thursday, February 27, 2014,

Issue with spaces in diffs' paths

2014-02-27 Thread Marek Dorobisz
Hello, I have a git repository and in paths there are spaces. Unfortunately Reviewboard ommits all before last space and tries to generate diffs with invalid paths. Is there any solution for that? Example path: $/ProjectName/release/some folder with spaces in name/anotherFolder/files.cs

Re: rbt post ignoring PUBLISH in .reviewboardrc?

2014-02-27 Thread David Trowbridge
Lee, Looks like there's a bug. Can you file this on the google code issue tracker? Thanks! -David On Thu, Feb 27, 2014 at 3:48 AM, Lee Laborczfalvi labor...@gmail.comwrote: I've just switched over to use rbt instead of post-review, but I'm finding that it's ignoring the PUBLISH=1 directive

Re: Issue 1986 in reviewboard: CVS repository(pserver) addition fails

2014-02-27 Thread reviewboard
Comment #15 on issue 1986 by rich...@163.com: CVS repository(pserver) addition fails http://code.google.com/p/reviewboard/issues/detail?id=1986 Do you have a data for the fix in 1.7.x branch? I want to use ReviewBoard to do code review in CVS environment, but it doesn't work in 1.7.21

Re: Issue 1986 in reviewboard: CVS repository(pserver) addition fails

2014-02-27 Thread reviewboard
Comment #16 on issue 1986 by trowb...@gmail.com: CVS repository(pserver) addition fails http://code.google.com/p/reviewboard/issues/detail?id=1986 If you'd like, you could make the equivalent changes in the code on your system:

Re: Issue 1986 in reviewboard: CVS repository(pserver) addition fails

2014-02-27 Thread reviewboard
Comment #17 on issue 1986 by rich...@163.com: CVS repository(pserver) addition fails http://code.google.com/p/reviewboard/issues/detail?id=1986 I modified and add function check_repository according to the above url. but indicate that six variable is not found. and then I added from