[ovs-dev] DPDK Merge Repo

[Darrell Ball]

-----Original Message-----
From: Ben Pfaff <blp at ovn.org>
Date: Wednesday, August 2, 2017 at 10:54 AM
To: Darrell Ball <dball at vmware.com>
Cc: "dev at openvswitch.org" <dev at openvswitch.org>
Subject: Re: [ovs-dev] DPDK Merge Repo

    On Wed, Aug 02, 2017 at 03:18:51PM +0000, Darrell Ball wrote:
    > 
    > 
    > -----Original Message-----
    > From: Ben Pfaff <blp at ovn.org>
    > Date: Wednesday, August 2, 2017 at 8:09 AM
    > To: Darrell Ball <dball at vmware.com>
    > Cc: "dev at openvswitch.org" <dev at openvswitch.org>
    > Subject: Re: [ovs-dev] DPDK Merge Repo
    > 
    >     On Wed, Aug 02, 2017 at 06:03:07AM +0000, Darrell Ball wrote:
    >     > Hi All
    >     > 
    >     > As mentioned before, I am using a repo for DPDK patch merging.
    >     > The repo is here:
    >     > https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_darball_ovs_&d=DwIBAg&c=uilaK90D4TOVoH58JNXRgQ&r=BVhFA09CGX7JQ5Ih-uZnsw&m=d2Eq2KJ8dkMVtpr5NXUPH0C4vP6QW4rdUIuQtHu-G9w&s=w-zS9fjLeXQTjM9hMFh2hVC8TgWzKjR3WNk6UrrrpMo&e= 
    >     
    >     Hi Darrell.  Thanks for working on this.
    >     
    >     When a patch is applied to a repo, for submission, the person who
    >     applies it needs to add a Signed-off-by.  May I assume that you intended
    >     to sign off each of these patches?  If so, then I can add that as part
    >     of applying them to the repo.
    > 
    > Yes, please, go ahead.
    > I debated this question myself in this particular context; now it is clear – thanks.
    >     
    >     (At the same time, when you sign off on a patch, you should remove any
    >     Acked-by for the patch, since the ack is implied.)
    > 
    > Right, thanks
    
    I made those changes and applied these patches to master.  I didn't
    apply them with a merge commit since I had to change them a bit.

Thanks Ben

Darrell