Doing chained diffs w/ Reviewboard

[David Cheney]

Also, be watchful for the other reviewboard footgun, paged diffs.

Reviewboard pages large reviews, so if you're used to thinking 'phew,
i've gotten to the end of the page, i'm done", check again, there
maybe a surprise waiting for you at the bottom of the page.

On Thu, Sep 18, 2014 at 9:03 PM, David Cheney
<david.cheney at canonical.com> wrote:
> +1 to that
>
> On Thu, Sep 18, 2014 at 7:53 PM, Adam Collard
> <adam.collard at canonical.com> wrote:
>> On 18 September 2014 10:49, John Meinel <john at arbash-meinel.com> wrote:
>>>
>>> Has anyone succeeded in getting this to work?
>>>
>>> The steps I tried to do were:
>>>
>>>  git co master
>>>  git pull upstream master
>>>  git co base-branch
>>>  git diff master... > base.diff
>>>  git co dependent-branch
>>>  git diff master... > dependent.diff
>>>  git merge-base master HEAD > remember-this-rev
>>>
>>> And then put the "dependent.diff" into the "Diff: *", and then the
>>> "base.diff" into "Parent Diff:" and then 'remember-this-rev' into the Base
>>> Commit ID.
>>>
>>> I also tried putting "git merge-base master base-branch" as the Base
>>> Commit ID.
>>
>>
>> This makes me think you're using the UI to do this.
>>
>> Let me repeat my Public Safety Announcement: Do NOT use ReviewBoard's UI for
>> uploading diffs. Please for $deity's sake use rbt post.
>>
>> https://www.reviewboard.org/docs/rbtools/0.6/rbt/commands/post/#distributed-version-control-systems
>>
>> --
>> Juju-dev mailing list
>> Juju-dev at lists.ubuntu.com
>> Modify settings or unsubscribe at:
>> https://lists.ubuntu.com/mailman/listinfo/juju-dev
>>