P4Exception in P4API.P4Adapter.run( describe -s ) when doing post-review

2009-05-28 Thread ajwicker
Hi, I’m trying to bring review board to run together with perforce for some time now, but never managed it to load up a changelist to the review board server. I’m using the post-review tool like this: python.exe c:/python25/scripts/post-review 440605 --debug debug.htm (The htm is attached

Re: P4Exception in P4API.P4Adapter.run( describe -s ) when doing post-review

2009-05-28 Thread Christian Hammond
I don't know why, but I can never seen to view attached .htm files on googlegroups. Can you e-mail me this directly as an attachment? Thanks, Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.review-board.org VMware, Inc. - http://www.vmware.com On Thu, May 28,

Upgrade to Review Board 1.0 rc 2

2009-05-28 Thread kunjal.par...@gmail.com
Hello, I am new to ReviewBoard and our company (Broadcom) use ReviewBoard for code review. I plan to upgrade to rc2 and want to make sure my steps are correct: I do not know how to get rb-site tool. Also, once I upgrade to new version, Do I move to new database or existing database and how this

Re: Upgrade to Review Board 1.0 rc 2

2009-05-28 Thread Christian Hammond
Hi, I assume you're currently using an SVN install? What version, and how old is it? Also, what version of what distribution are you using? You'll basically be following the installation instructions for a brand new install, but will specify your existing database information. I assume this is

Re: Upgrade to Review Board 1.0 rc 2

2009-05-28 Thread kunjal.par...@gmail.com
Thanks for so fast reply.I was not expecting it.. Chris. We are using Perforce. We are using ReviewBoard from last one year and it come to my control just yesterday. I am not sure what version is installed but it would be nice if there is a way to check which review board version is installed.

Re: Review Board crashed with the error Table is marked as crashed and should be repaired

2009-05-28 Thread David Trowbridge
If you're the administrator, you need to use the mysql administration programs to repair your DB. If you're not, you need to ask your admin. -David On Thu, May 28, 2009 at 4:44 PM, kunjal.par...@gmail.com kunjal.par...@gmail.com wrote: OperationalError at /dashboard/ (1194, Table

Re: MEDIA_SERIAL not defined

2009-05-28 Thread Will
Same symptoms and javascript console error on Firefox 3.0.10. IE8 doesn't seem to function any better (but I can't find any error messages...). On May 27, 9:14 pm, Christian Hammond chip...@chipx86.com wrote: Can you test with something before Firefox 3.5? Wondering if it's a bug with 3.5b4.

Re: MEDIA_SERIAL not defined

2009-05-28 Thread Christian Hammond
Can you send me personally an HTML of the dashboard or some other page? Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.review-board.org VMware, Inc. - http://www.vmware.com On Thu, May 28, 2009 at 9:51 PM, Will will.le...@gmail.com wrote: Same symptoms and

Issue 1071 in reviewboard: Review Board does not handle files with ^M characters at end of line.

2009-05-28 Thread codesite-noreply
Updates: Status: Fixed Owner: chipx86 Labels: -Priority-Medium Priority-High Milestone-Release1.0 Component-DiffViewer Component-RBTools Comment #1 on issue 1071 by chipx86: Review Board does not handle files with ^M characters at end of line.

Issue 840 in reviewboard: post-review can't diff correctly Perforce paths with spaces?

2009-05-28 Thread codesite-noreply
Updates: Status: Fixed Owner: chipx86 Labels: -Priority-Medium -Component-Scripts Priority-High Component-RBTools Milestone-Release1.0 Comment #5 on issue 840 by chipx86: post-review can't diff correctly Perforce paths with spaces?

Issue 132 in reviewboard: Include source context in review e-mails

2009-05-28 Thread codesite-noreply
Comment #4 on issue 132 by scott.quesnelle: Include source context in review e-mails http://code.google.com/p/reviewboard/issues/detail?id=132 A number of developers here commented on the same thing. I have modified the review_email.txt template in my local installation to include the first