Sorry for the late reply. I've been on vacation.
I'm not sure I'm fully understanding the problem you're hitting. Let me see
if I have this right. You have a changelist with some code in it. You sync,
new changes are pulled in. You post the new changeset to Review Board, and
the diff is including the things that came from the sync?
I've never seen this behavior, and use Review Board with Perforce almost
daily. Posting a new review request *should* include only what changed for
When you say a "get latest", is this 'p4 sync', or something else?
This sounds like this is a client-side problem, where the changes aren't
resolved properly or the metadata isn't being updated properly, causing the
revision for the files to be incorrect.
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com
On Mon, May 7, 2012 at 8:39 AM, Robert Dailey <rcdailey.li...@gmail.com>wrote:
> Can someone help me out here?
> On Wed, May 2, 2012 at 1:37 PM, Robert Dailey <rcdailey.li...@gmail.com>wrote:
>> So I'm using Perforce and I upload my initial reviews (as well as follow
>> up changes and updates) using this command. Note that this is a "Custom
>> Tool" in P4V:
>> %C --server=http://reviewboard.company.com --p4-client=$c --p4-port=$p
>> --username=user --password=password
>> %C is the changelist number in Perforce
>> $c is the current workspace
>> $p is the current port
>> The problem I'm experiencing is updates to code in my changelist being
>> included in my diffs in reviewboard. So suppose I have files checked out in
>> my changelist. If I do a "get latest" from Perforce, and some of those
>> checked out files get changed, and I resolve conflicts, those updates I
>> pulled down will now show up in my diff. Reviewboard should not be showing
>> these changes in my diff, in other words, Reviewboard needs to use the
>> latest base of each file when diffing, which it does not. It continues to
>> use the same revisions of the files that it originally used when the review
>> was first created.
>> Is there a fix for this?
> Want to help the Review Board project? Donate today at
> Happy user? Let us know at http://www.reviewboard.org/users/
> To unsubscribe from this group, send email to
> For more options, visit this group at
Want to help the Review Board project? Donate today at
Happy user? Let us know at http://www.reviewboard.org/users/
To unsubscribe from this group, send email to
For more options, visit this group at