> -----Original Message----- > From: Aviad Yehezkel [mailto:[email protected]] > Sent: Monday, October 16, 2017 11:42 AM > To: Gonzalez Monroy, Sergio <[email protected]>; > [email protected]; De Lara Guarch, Pablo <[email protected]>; > [email protected] > Cc: [email protected]; [email protected]; > [email protected]; Nicolau, Radu <[email protected]>; > Doherty, Declan <[email protected]>; [email protected]; > [email protected]; [email protected] > Subject: Re: [dpdk-dev][PATCH 05/11] examples/ipsec-secgw: Fixed > transport > > > On 10/16/2017 12:30 PM, Sergio Gonzalez Monroy wrote: > > On 14/10/2017 14:27, [email protected] wrote: > >> From: Aviad Yehezkel <[email protected]> > >> > >> Seems like transport was broken for a long time > > > > Commit message needs to be improved. Just mentioned what is wrong or > > how do you fix it. > > Given that it is a fix, you should start the commit title with "fix > > ..." then also add the 'fixes' line with commit that added the bug. > > That way you can easily see since when it was introduced. > > > > Thanks, > > Sergio > Will create such commit messages for future fixes as you instructed above. > I will remove this patch from next version of fixes since there is a similar > fix > already provided by Tomasz Duszynski which I will review. > > Thanks, > Aviad.
Hi Aviad, Will you send a v2 of this patchset soon? Thanks, Pablo
