Cookies must be enabled to login to review board

2010-12-10 Thread D a s h
I have setup a review board server on RedHat linux. I am able to login to this review board site using FireFox. But the same is not possible using IE8 browser. I tried setting all possible configuration for cookes (Privacy settings - Accept All cookies). Even then, no luck. I have seen similar

Re: Aggregated diffs and added files

2010-12-10 Thread Sasha O
Vincent -- Thanks for your reply. Before writing to the group I have tried post-review with the similar results. But now a strange thing happened -- I cannot reproduce this problem. Both post-review and manual upload work fine now on this changeset (hope it's the same changeset). I will keep

Unable to do HTTP Post to Diff List resource, {fields: {path: [This field is required]}, stat: fail, err: {msg: One or more fields had errors, code: 105}}

2010-12-10 Thread Dlite
I am using Java and HttpClient to interface with the reviewboard web API.I have created a new review request draft using HTTP Post, but I am stuck with the diff part. From what I read (http://www.reviewboard.org/docs/manual/1.5/webapi/ 2.0/resources/diff-list/), to create a new diff, I need to

How to create the diff part of review request

2010-12-10 Thread Dlite
I have a request draft created using the web API. I need to create the diff file and link the diff to my request draft. I am not sure how to do it after I read through the reviewboard documentation. I am using Java and HttpClient to interface with reviewboard. I have svn repository. It doesn't

Re: Issue 839 in reviewboard: Option to convert tabs in the diff to (n) spaces

2010-12-10 Thread reviewboard
Comment #13 on issue 839 by d...@thompsonhome.org: Option to convert tabs in the diff to (n) spaces http://code.google.com/p/reviewboard/issues/detail?id=839 Given that the purpose of this entire system is to facilitate code reviews I question why this feature/bug is low priority. Doing

Re: Issue 839 in reviewboard: Option to convert tabs in the diff to (n) spaces

2010-12-10 Thread reviewboard
Comment #14 on issue 839 by paul.dubois: Option to convert tabs in the diff to (n) spaces http://code.google.com/p/reviewboard/issues/detail?id=839 @13, No judgement on your priority argument but modifying the .css is a usable workaround to get server-side tab settings. I can confirm it