[ovs-dev] DPDK Merge Repo

Ben Pfaff blp at ovn.org
Wed Aug 2 17:54:57 UTC 2017


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.


More information about the dev mailing list