post-review login issue

I was trying to create a post review request with the command on Windows: post-review 388600 388600 is the changelist # of perforce. The script was able generate the diff but failed on logon the review board server, its url is: http://b00237d4f5dc7/reviews/vtm/ The error message

Re: post-review login issue

Hammond - chip...@chipx86.com Review Board -http://www.review-board.org VMware, Inc. -http://www.vmware.com On Sat, Sep 19, 2009 at 9:57 PM, Hui Lin hui@bankofamerica.com wrote: I was trying to create a post review request with the command on Windows:    post-review 388600 388600

Re: post-review login issue

Is there a debug flag on the review board server which can be turned on to trace what is happening? On Sep 20, 7:18 pm, Hui Lin hui@bankofamerica.com wrote: http://b00237d4f5dc7/reviews/vtm/; redirect me to the dashboard. Going to http://b00237d4f5dc7/reviews/vtm/api/json/accounts/login

Re: post-review login issue

it's not the Review Board one. Christian -- Christian Hammond - chip...@chipx86.com Review Board -http://www.review-board.org VMware, Inc. -http://www.vmware.com On Sun, Sep 20, 2009 at 5:18 PM, Hui Lin hui@bankofamerica.com wrote: http://b00237d4f5dc7/reviews/vtm/; redirect me

Re: post-review login issue

, Sep 21, 2009 at 10:55 AM, Hui Lin hui@bankofamerica.com wrote: Thanks Christian. I resolved the issue by install the 1.0.3 version and re-create the site. Now post-review is able to logon the reviewer board server, but gets problem of understanding the repository_path of the SCM

Re: post-review login issue

, 2009 at 2:14 PM, Hui Lin hui@bankofamerica.com wrote: Yes, I solved this issue by putting what returned from p4 info exactly to the repository setup. My local p4 was using an alias of what from p4 info. Then, I bumped against another issue Perforce password (P4PASSWD) invalid or unset

Re: post-review login issue

PM, Hui Lin hui@bankofamerica.com wrote: I believe uploading the diff is just a reviewer board function, which doesn't seem to have anything to do with to p4. On Sep 21, 7:15 pm, Christian Hammond chip...@chipx86.com wrote: Hi, I don't personally have a good answer to the P4PASSWD

Re: post-review login issue

great. On Sep 23, 12:00 pm, Hui Lin hui@bankofamerica.com wrote: Hi, Dana: I saw your posts are rendered at full length each line, but mine are wrapped at 70-80 characters, which makes them look quite different from what I originally posted. What is the trick? Thanks On Sep 23, 11:28

Question about setting up Default Reviewers

If we want person A to review all the changes made to any class of in directory D, but also want person B to review changes made to a particular class C1 in D, how should we define the default reviewers? Would the followings work? Or only the first matched reviewer will be picked? * Default

Question about customizing the diff view

I noticed that the lines in the diff view are automatically wrapped at the boundary of the browser windows. Is there a way to turn off the wrapping, and allow people to use horizontal scroll bar to control what is shown on the screen? --~--~-~--~~~---~--~~ You

Re: Question about customizing the diff view

://www.review-board.org VMware, Inc. -http://www.vmware.com On Fri, Sep 25, 2009 at 8:31 AM, Hui Lin hui@bankofamerica.com wrote: I noticed that the lines in the diff view are automatically wrapped at the boundary of the browser windows. Is there a way to turn off the wrapping, and allow