About Reviewboard “ShipIt” operations

2010-03-17 Thread JohnHenry
HI, all I am using RB V1.0.5.1; One of our users had problems on reviews with FireFox3.6; As a Reviewer, He can't operate ShipIt. When he reviewed and click Ship It and then publish, The result is just a publish of review without shipit tag and even without any text of Reviews. Others user with

Re: About Reviewboard “ShipIt” operations

2010-03-17 Thread Christian Hammond
Is this with your version containing custom modifications? We use Firefox for development and have had no problems with this. Nobody has reported any such problem with 1.0.5.1 either. I can't think what would cause that. Christian -- Christian Hammond - chip...@chipx86.com Review Board -

500 Server Error when attempting to post

2010-03-17 Thread Kyle Hayes
I'm trying to post my first review to our newly installed review board: $ post-review 656260 --p4-client=p4-tp-nd Unable to access http://reviews.poseidon/api/json/reviewrequests/new/. The host path may be invalid HTTP Error 500: Internal Server Error When I do post-review -d, I see the

How to allow access on reviews per repository

2010-03-17 Thread Mihai
Hello, First of all thank you for the great app. Second, I am in need of an advice/info. I have more repositories for different teams and I would like the users to have access only on reviews from a specific repository. Is this possible? Thank you again, Mihai -- Want to help the Review Board

Re: How to allow access on reviews per repository

2010-03-17 Thread Stephen Gallagher
On 03/17/2010 10:48 AM, Mihai wrote: Hello, First of all thank you for the great app. Second, I am in need of an advice/info. I have more repositories for different teams and I would like the users to have access only on reviews from a specific repository. Is this possible? Thank you

Re: 500 Server Error when attempting to post

2010-03-17 Thread Christian Hammond
Which versions of Review Board and RBTools? I assume this is using Perforce? Can you run with --output-diff and see if you get a meaningful diff back? Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Wed,

Re: 500 Server Error when attempting to post

2010-03-17 Thread Christian Hammond
Can you try a nightly of RBTools? This might be related to a bug we've fixed. What happens if you take that diff and upload through the web UI? Christian On Wednesday, March 17, 2010, Hayes, Kyle kyle.ha...@disney.com wrote: RB: 1.0.5.1 Tools: 0.2rc1 Yeah, we’re on Perforce

WindowsError: [Error 2] The system cannot find the file specified

2010-03-17 Thread RShelley
Hey folks, I'm evaluating ReviewBoard and trying to get it to work with our Perforce SCM but we're getting an error when I attempt to run the post-review command: (I've abridged the paths with ellipses for brevity) Error: --- C:\p4-tp-1.02post-review

Re: WindowsError: [Error 2] The system cannot find the file specified

2010-03-17 Thread Christian Hammond
Hi, You need to have GNU diff in your system path somewhere. You can get this at: http://gnuwin32.sourceforge.net/packages/diffutils.htm We're going to make this error more clear in an upcoming version. Christian -- Christian Hammond - chip...@chipx86.com Review Board -

RE: 500 Server Error when attempting to post

2010-03-17 Thread Hayes, Kyle
Ok, I got the nightly of RBTools and am now getting the following error: $ post-review 656260 --p4-client=p4-tp-nd -d svn info p4 info repository info: Path: #..com:1666, Base path: None, Supports changesets: True Generating diff for changenum 656260 p4 describe -s 656260 Processing

Re: 500 Server Error when attempting to post

2010-03-17 Thread Christian Hammond
That absolutely shouldn't be happening. Can you turn on logging on the Review Board server (you may have to restart Apache after -- this bug is being fixed) and see what the log file says? There should be a stack trace in there that I can look at. Christian -- Christian Hammond -

Re: WindowsError: [Error 2] The system cannot find the file specified

2010-03-17 Thread RShelley
Thanks Christian, that helped, and thanks for the extremely prompt response! -Ryan On Mar 17, 3:01 pm, Christian Hammond chip...@chipx86.com wrote: Hi, You need to have GNU diff in your system path somewhere. You can get this at: http://gnuwin32.sourceforge.net/packages/diffutils.htm

RE: 500 Server Error when attempting to post

2010-03-17 Thread Hayes, Kyle
Ok, I enabled logging (and chown the dir for www-data) and restarted, then re-ran my command, but the logs don't appear to be getting outputted to it. Kyle Hayes Sr. Web Developer | Disney Family Disney Interactive Media Group 818-623-3847 AIM/YIM/gTalk: mrkylehayes It's kind of fun to do

Issue 1547 in reviewboard: Group count on My Dashboard are incorrect or misleading.

2010-03-17 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1547 by tehill: Group count on My Dashboard are incorrect or misleading. http://code.google.com/p/reviewboard/issues/detail?id=1547 What version are you running? 1.5 beta 1 What's the URL of the page containing the

Issue 1548 in reviewboard: updating a review request fails with UnboundLocalError: local variable 'rsp' referenced before assignment

2010-03-17 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1548 by irwan.djajadi: updating a review request fails with UnboundLocalError: local variable 'rsp' referenced before assignment http://code.google.com/p/reviewboard/issues/detail?id=1548 version: RBTools-0.2rc1-py2.6.egg

Issue 1549 in reviewboard: The email-address 'myn...@localhost' is not considered valid.

2010-03-17 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1549 by holmstrom.daniel: The email-address 'myn...@localhost' is not considered valid. http://code.google.com/p/reviewboard/issues/detail?id=1549 *NOTE: Do not post confidential information in this bug report.* What

Re: Issue 949 in reviewboard: post-review perforce HTTP Error 500: Internal Server Error

2010-03-17 Thread reviewboard
Comment #3 on issue 949 by mrkylehayes: post-review perforce HTTP Error 500: Internal Server Error http://code.google.com/p/reviewboard/issues/detail?id=949 Did you only have to install p4python to get it to work? Or did you have to configure it somehow afterwards? -- You received this

Issue 1550 in reviewboard: test

2010-03-17 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1550 by Saed.AlQannas: test http://code.google.com/p/reviewboard/issues/detail?id=1550 *NOTE: Do not post confidential information in this bug report.* What version are you running? What's the URL of the page containing

Issue 1551 in reviewboard: post-review: need an option to not save the cookie

2010-03-17 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1551 by nahor.j: post-review: need an option to not save the cookie http://code.google.com/p/reviewboard/issues/detail?id=1551 What version are you running? post-review 0.8 Please provide any additional information below.

Re: Issue 734 in reviewboard: Site Settings change requires Apache restart

2010-03-17 Thread reviewboard
Comment #11 on issue 734 by chipx86: Site Settings change requires Apache restart http://code.google.com/p/reviewboard/issues/detail?id=734 Yeah, it's definitely a bug in 1.5 beta 1, but I'd like to see if people still hit it with the latest nightlies of both Review Board and Djblets. --

Re: Issue 1550 in reviewboard: test

2010-03-17 Thread reviewboard
Updates: Status: Spam Comment #1 on issue 1550 by chipx86: test http://code.google.com/p/reviewboard/issues/detail?id=1550 (No comment was entered for this change.) -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this

Re: Issue 1551 in reviewboard: post-review: need an option to not save the cookie

2010-03-17 Thread reviewboard
Updates: Status: Confirmed Labels: -Type-Defect -Priority-Medium Type-Enhancement Priority-Low Milestone-RBTools-Release1.0 Component-RBTools Comment #1 on issue 1551 by chipx86: post-review: need an option to not save the cookie

Re: Issue 1547 in reviewboard: Group count on My Dashboard are incorrect or misleading.

2010-03-17 Thread reviewboard
Updates: Status: Fixed Owner: chipx86 Labels: Component-Dashboard Milestone-Release1.5 Comment #1 on issue 1547 by chipx86: Group count on My Dashboard are incorrect or misleading. http://code.google.com/p/reviewboard/issues/detail?id=1547 This is fixed in the

Re: Issue 1510 in reviewboard: Exception thrown during post-review -- 'rsp' is used before being defined

2010-03-17 Thread reviewboard
Comment #5 on issue 1510 by chipx86: Exception thrown during post-review -- 'rsp' is used before being defined http://code.google.com/p/reviewboard/issues/detail?id=1510 Issue 1548 has been merged into this issue. -- You received this message because you are listed in the owner or CC fields

Re: Issue 1549 in reviewboard: The email-address 'myn...@localhost' is not considered valid.

2010-03-17 Thread reviewboard
Comment #3 on issue 1549 by chipx86: The email-address 'myn...@localhost' is not considered valid. http://code.google.com/p/reviewboard/issues/detail?id=1549 Thanks. That's good feedback. We should make a decision as to how much validation we want to do, and where. Are there other things

Issue 1553 in reviewboard: patch crash in reviewboard, post-review's difflib_unified_diff() fails to handle missing newline terminated file

2010-03-17 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1553 by hortonlkh: patch crash in reviewboard, post-review's difflib_unified_diff() fails to handle missing newline terminated file http://code.google.com/p/reviewboard/issues/detail?id=1553 *NOTE: Do not post confidential

Issue 1554 in reviewboard: Replace Summary: with Review # -

2010-03-17 Thread reviewboard
Status: New Owner: Labels: Type-Enhancement Priority-Medium New issue 1554 by nahor.j: Replace Summary: with Review # - http://code.google.com/p/reviewboard/issues/detail?id=1554 What version are you running? Review Board 1.5b1 Describe the enhancement and the motivation for it. In

Issue 1555 in reviewboard: Reviewers list is lost when the review is re-posted.

2010-03-17 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1555 by ksangani: Reviewers list is lost when the review is re-posted. http://code.google.com/p/reviewboard/issues/detail?id=1555 *NOTE: Do not post confidential information in this bug report.* What version are you

Issue 1556 in reviewboard: post-review waits for password if the Subversion requires authentication

2010-03-17 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1556 by nahor.j: post-review waits for password if the Subversion requires authentication http://code.google.com/p/reviewboard/issues/detail?id=1556 What version are you running? post-review 0.8 Please provide any

Issue 1557 in reviewboard: post-review crashed if the home directory is not writable

2010-03-17 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1557 by nahor.j: post-review crashed if the home directory is not writable http://code.google.com/p/reviewboard/issues/detail?id=1557 What version are you running? post-review 0.8 What steps will reproduce the problem? 1.