Eclipse SVN Workflow

2012-06-21 Thread Jenny Hong
How are people dealing with the workflow in the ReviewBoard plug-in for Eclipse? Some problems we are encountering are - doing separate posts for every review - changing workspaces to do pre-commit code reviews, which are expensive. Thanks! -- Want to help the Review Board project?

Re: Eclipse SVN Workflow

2012-06-21 Thread Robert Munteanu
Hi Jenny, On Thursday, June 21, 2012 8:47:11 PM UTC+3, Jenny Hong wrote: How are people dealing with the workflow in the ReviewBoard plug-in for Eclipse? Some problems we are encountering are - doing separate posts for every review How would you propose streamlining the process of

Re: Eclipse SVN Workflow

2012-06-21 Thread Jenny Hong
Hi Robert, Sorry, I was a little unclear with my first question; I meant that we need to do separate reviews for each project. Is there some way to submit a single review that covers multiple projects? With regard to the workflow, there are a few options, but all of them have significant

Re: Eclipse SVN Workflow

2012-06-21 Thread Chris Tooley
You can tag local revision sets of your changes in Eclipse thereby creating a reference point to diff against. By combining that with subversion revision numbers or labels you should be able you get what I think you're describing. On Jun 21, 2012 6:57 PM, Jenny Hong thadel...@gmail.com wrote: Hi

Re: Diff fie size limited to 1 MB

2012-06-21 Thread Christian Wuerdig
Hi Christian, we just hit that issue here by some reviews missing the diff files. In general this is caused when someone works on a feature branch in the repository and then merges in the changes from the trunk. Although it is supposed to happen regularly sometimes there can be two or three

Re: Diff fie size limited to 1 MB

2012-06-21 Thread Christian Wuerdig
Sorry, ignore that, just realized that it is configurable since 1.6.7 On Friday, June 22, 2012 3:51:56 PM UTC+12, Christian Wuerdig wrote: Hi Christian, we just hit that issue here by some reviews missing the diff files. In general this is caused when someone works on a feature branch in

Re: Diff fie size limited to 1 MB

2012-06-21 Thread Christian Hammond
Hi, Upgrade Review Board. The limit is configurable and disabled by default. Christian On Jun 21, 2012, at 20:51, Christian Wuerdig christian.wuer...@gmail.com wrote: Hi Christian, we just hit that issue here by some reviews missing the diff files. In general this is caused when

multiple review request for same change list do we keep count in RB database

2012-06-21 Thread Nilesh Jaiswal
Hi All, If we post review request multiple time for the same change-list, we could see same review id, but do we keep such count in the database of RB for multiple review request for same changelist. if yes then can you please point me the table for the same or any other way to get the such

Re: Issue 2643 in reviewboard: Changing to github hosted service after upgrading to 1.6.9 fails with Unable to link the account: HTTP Error 201: Created

2012-06-21 Thread reviewboard
Comment #9 on issue 2643 by chris.a@gmail.com: Changing to github hosted service after upgrading to 1.6.9 fails with Unable to link the account: HTTP Error 201: Created http://code.google.com/p/reviewboard/issues/detail?id=2643 It appears this is biting other python clients. Here's an

Issue 2647 in reviewboard: P4 charset setting is ignored when making a connection

2012-06-21 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2647 by raya...@gmail.com: P4 charset setting is ignored when making a connection http://code.google.com/p/reviewboard/issues/detail?id=2647 What version are you running? 1.6.9 What's the URL of the page containing the

Re: Issue 256 in reviewboard: Add/Delete/Edit/integrate information not shown for binary files.

2012-06-21 Thread reviewboard
Comment #7 on issue 256 by william@gmail.com: Add/Delete/Edit/integrate information not shown for binary files. http://code.google.com/p/reviewboard/issues/detail?id=256 I'd prefer an option to not show the binary files at all or summarize them separately from the readable code