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.che...@canonical.com> wrote: > +1 to that > > On Thu, Sep 18, 2014 at 7:53 PM, Adam Collard > <adam.coll...@canonical.com> wrote: >> On 18 September 2014 10:49, John Meinel <j...@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@lists.ubuntu.com >> Modify settings or unsubscribe at: >> https://lists.ubuntu.com/mailman/listinfo/juju-dev >> -- Juju-dev mailing list Juju-dev@lists.ubuntu.com Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/juju-dev
