post-review from windows fails

2011-03-15 Thread rstm
Hello, Until now I used post-review (RBTools 0.2) from my mac and I can push diffs without any trouble to a ReviewBoard 1.5.2. Now I started using a Windows client and get strange errors when trying to push a diff. RBTools 0.3.2: I cannot push a diff, I just get the following error message.

Reviewing diffs of large merges

2011-03-15 Thread Bobman
At my company we have a recurring issue (we use CVS) when we merge a branch back to the trunk, where (if it has been several months) the changes and conflicts need to be reviewed by many people. Ideally, we would run a diff between the two branches and publish that for others to review, but a

Re: post-review from windows fails

2011-03-15 Thread Christian Hammond
Hi Ruben, Are you behind a proxy server? If so, it's possible that post-review is picking up the proxy settings on Windows, and it's preventing it from accessing the site correctly. With 0.3.2, can you run with --debug and provide the output? Christian -- Christian Hammond -

Re: LDAP and Non Ascii Characters

2011-03-15 Thread Eric Johnson
Could be a browser configuration issue too. Did you ask the person to try from a different browser? Eric On Mar 14, 2011, at 4:45 AM, daniel.j.la...@googlemail.com daniel.j.la...@googlemail.com wrote: I am using LDAP to authenticate users in Review board - this obviously means I have little

Re: Reviewing diffs of large merges

2011-03-15 Thread David Trowbridge
Review Board can deal with large diffs, but I don't think there's really much in the way of helping reviewers deal with it. The only help I can think of is that it will split it across pages in order to avoid having giant web pages to look at (by default 20 files to a page). If there are specific

Issue 2025 in reviewboard: Server error 404 when adding comments to screenshot

2011-03-15 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2025 by marco.zi...@gmail.com: Server error 404 when adding comments to screenshot http://code.google.com/p/reviewboard/issues/detail?id=2025 * NOTE: Do not post confidential information in this bug report. * * If

Re: Issue 2025 in reviewboard: Server error 404 when adding comments to screenshot

2011-03-15 Thread reviewboard
Updates: Status: Confirmed Labels: Component-Screenshots Milestone-Release1.5.x Comment #1 on issue 2025 by chip...@gmail.com: Server error 404 when adding comments to screenshot http://code.google.com/p/reviewboard/issues/detail?id=2025 It looks like somehow, the coordinates

Issue 2026 in reviewboard: Add Change descriptions to the Web API

2011-03-15 Thread reviewboard
Status: New Owner: Labels: Type-Enhancement Priority-Medium New issue 2026 by robert.munteanu: Add Change descriptions to the Web API http://code.google.com/p/reviewboard/issues/detail?id=2026 What version are you running? 1.5.4 Describe the enhancement and the motivation for it. Change

Re: Issue 2025 in reviewboard: Server error 404 when adding comments to screenshot

2011-03-15 Thread reviewboard
Comment #2 on issue 2025 by marco.zi...@gmail.com: Server error 404 when adding comments to screenshot http://code.google.com/p/reviewboard/issues/detail?id=2025 This rings a bell of something that might be related: when I create a comment on a screenshot and I do actually succeed with

Re: Issue 2025 in reviewboard: Server error 404 when adding comments to screenshot

2011-03-15 Thread reviewboard
Comment #3 on issue 2025 by marco.zi...@gmail.com: Server error 404 when adding comments to screenshot http://code.google.com/p/reviewboard/issues/detail?id=2025 Some more test data: when opening the same review in ie8 the regions are different from the ones that are shown in chrome. When

Issue 2027 in reviewboard: can't comment to diff

2011-03-15 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2027 by ahmed.ab...@gmail.com: can't comment to diff http://code.google.com/p/reviewboard/issues/detail?id=2027 * NOTE: Do not post confidential information in this bug report. * * If you need immediate support, please

Re: Issue 2026 in reviewboard: Add Change descriptions to the Web API

2011-03-15 Thread reviewboard
Updates: Status: Confirmed Owner: chip...@gmail.com Labels: -Priority-Medium Priority-High Milestone-Release1.6 Component-API Comment #1 on issue 2026 by chip...@gmail.com: Add Change descriptions to the Web API

Re: Issue 1903 in reviewboard: [Discarded] label could be red to diferentiate from Submitted ones

2011-03-15 Thread reviewboard
Updates: Status: Confirmed Labels: -Priority-Medium Priority-Low EasyFix Comment #2 on issue 1903 by chip...@gmail.com: [Discarded] label could be red to diferentiate from Submitted ones http://code.google.com/p/reviewboard/issues/detail?id=1903 (No comment was entered for

Re: Issue 1893 in reviewboard: Links in summary and description fields

2011-03-15 Thread reviewboard
Updates: Status: NeedInfo Comment #1 on issue 1893 by chip...@gmail.com: Links in summary and description fields http://code.google.com/p/reviewboard/issues/detail?id=1893 They are. What have you tried? Can you check if there are JavaScript errors? -- You received this message

Re: Issue 1612 in reviewboard: WebAPI searching by bug

2011-03-15 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 1398 Comment #4 on issue 1612 by chip...@gmail.com: WebAPI searching by bug http://code.google.com/p/reviewboard/issues/detail?id=1612 (No comment was entered for this change.) -- You received this message because you are subscribed to the

Re: Issue 1398 in reviewboard: API to search reviews by bug ID and query review status

2011-03-15 Thread reviewboard
Comment #2 on issue 1398 by chip...@gmail.com: API to search reviews by bug ID and query review status http://code.google.com/p/reviewboard/issues/detail?id=1398 Issue 1612 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups

Re: Issue 1338 in reviewboard: Allow repository-id to be used as a parameter instead of repository-path

2011-03-15 Thread reviewboard
Updates: Status: Fixed Comment #3 on issue 1338 by chip...@gmail.com: Allow repository-id to be used as a parameter instead of repository-path http://code.google.com/p/reviewboard/issues/detail?id=1338 Support for this was committed a little while back, I think in 1.5.3. -- You

Re: Issue 2005 in reviewboard: IE support broken in 1.6beta1

2011-03-15 Thread reviewboard
Comment #5 on issue 2005 by d...@n-cube.org: IE support broken in 1.6beta1 http://code.google.com/p/reviewboard/issues/detail?id=2005 Yes there are some developer tools though they don't show quite the detail that Chrome and Firebug too. I think I have found the issue, though it's not clear

Re: Issue 1571 in reviewboard: Diff-Upload UI: Base Path field too narrow

2011-03-15 Thread reviewboard
Updates: Status: Fixed Comment #12 on issue 1571 by trowb...@gmail.com: Diff-Upload UI: Base Path field too narrow http://code.google.com/p/reviewboard/issues/detail?id=1571 Fixed in master (1.6) as a5a493e. Thanks! -- You received this message because you are subscribed to the