> > -----Original Message----- > > From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- > > boun...@openvswitch.org] On Behalf Of Stokes, Ian > > Sent: Thursday, June 20, 2019 11:53 AM > > To: Ilya Maximets <i.maxim...@samsung.com>; David Marchand > > <david.march...@redhat.com> > > Cc: ovs dev <d...@openvswitch.org> > > Subject: Re: [ovs-dev] [PATCH v2 2/2] travis: Make it possible to build > > against a dpdk branch. > > > > > On 19.06.2019 14:35, David Marchand wrote: > > > > > > > > > > > > On Wed, Jun 19, 2019 at 1:22 PM Ilya Maximets > <i.maxim...@samsung.com > > > <mailto:i.maxim...@samsung.com>> wrote: > > > > > > > > On 19.06.2019 10:26, David Marchand wrote: > > > > > Rework the build script so that we can pass branches and tags. > > > > > > > > > > With this, DPDK_VER can be passed as: > > > > > - a string starting with refs/ which is understood as a git > > > reference. > > > > > This triggers a git clone on DPDK_GIT (default value points > to > > > > > https://dpdk.org/git/dpdk) for a single branch pointing to > > this > > > > > reference (to save some disk), > > > > > - else, any other string which is understood as an official > > > release. > > > > > This triggers a tarball download on dpdk.org > > <http://dpdk.org>. > > > > > > > > > > Signed-off-by: David Marchand <david.march...@redhat.com > > > <mailto:david.march...@redhat.com>> > > > > > --- > > > > > Changelog since v1: > > > > > - removed (now unneeded) directory renames > > > > > - added a "git log" so that we have the current git revision > in > > > the logs > > > > > > > > > > --- > > > > > > > > > > > > Thanks! > > > > > > > > I tested this patch with: > > > > > > > > - DPDK=1 DPDK_GIT="https://dpdk.org/git/dpdk-stable"; > > > DPDK_VER="refs/heads/18.11" > > > > > > > > and > > > > > > > > - DPDK=1 DPDK_VER="refs/heads/master" > > > > > > > > Works fine. > > > > > > > > > > > > Thanks Ilya. > > > > > > > > I could have detailed my non-reg tests: > > > > - DPDK=1 > > > > - DPDK=1 DPDK_VER=18.11.2 > > > > - DPDK=1 DPDK_VER=refs/tags/v18.11.2 > > DPDK_GIT=http://dpdk.org/git/dpdk- > > > stable > > > > - DPDK=1 DPDK_VER=refs/heads/18.11 > DPDK_GIT=http://dpdk.org/git/dpdk- > > > stable > > > > > > > > > > > > > > > > So, I pushed this and the previous patch to master. > > > > > > > > > > > > Cool, so I suppose you will handle the changes on dpdk-latest, > right? > > > > > > Not sure. Ian managed these branches (hwol, latest) previously. > > > > > > Ian, will you rebase dpdk-latest onto current master? > > > > Yes, I'll look at this today. I know when I look last week there were a > > few conflicts to be resolved. So will sort these. > > Hi Ilya, just started looking at this again, are yu sure it's a rebase we > want here i.e. rebase dpdk-latest on master? > > Essentially we what I'm seeing is as the dpdk-latest changes are being > applied ontop of the least master history all the previous changes in > dpdk-latest have to re-applied include changes such as upgrade to 18.08 > etc. it's just making it a bit messy, especially with some of the HOWL > changes we've had on master. > > I'm also think would it not require a force push to the dpdk-latest > branch? The re-write of the commit history will change etc. > > I know the merge process wasn't ideal from a commit history POV but it did > avoid issues such as this in the past. What are your thoughts?
To provide a better example, what I mean would be if you look at the commit for moving to 18.11 on master, in the commit message we actually use commit IDs form commits to dpdk-latest to reference and help accredit the authorship of work. If we rebase dpdk-latest with master, these commit ID's change in the dpdk-latest branch, meaning in master we have incorrect commit ID etc. in the commit message. I guess that’s what I was trying to avoid with the rebase approach and hence why I had used the merge approach (similar to how we used to use dpdk-merge branches). Do you have a way around this? Thanks Ian > > Ian > > > > Thanks > > Ian > > > > > > > > Best regards, Ilya Maximets. > > _______________________________________________ > > dev mailing list > > d...@openvswitch.org > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
