Re: [dpdk-dev] [PATCH] scripts: check cc stable mailing list in commit

2017-01-16 Thread Yuanhan Liu
On Mon, Jan 16, 2017 at 03:26:03PM +0100, Thomas Monjalon wrote: > > > > Besides, if there is already an explicit way, why should we stick on the > > > > implicit way? > > > > > > Because the explicit way is not 100% proof. > > > > Yes, but I think it will be close to "100%" as time moves forward

Re: [dpdk-dev] [PATCH] scripts: check cc stable mailing list in commit

2017-01-16 Thread Thomas Monjalon
2017-01-16 19:19, Yuanhan Liu: > On Mon, Jan 16, 2017 at 11:37:34AM +0100, Thomas Monjalon wrote: > > 2017-01-16 17:51, Yuanhan Liu: > > > On Wed, Nov 30, 2016 at 02:54:14PM +, Ferruh Yigit wrote: > > > > On 11/21/2016 10:43 PM, Thomas Monjalon wrote: > > > > > Add a check for commits fixing a

Re: [dpdk-dev] [PATCH] scripts: check cc stable mailing list in commit

2017-01-16 Thread Yuanhan Liu
On Mon, Jan 16, 2017 at 11:37:34AM +0100, Thomas Monjalon wrote: > 2017-01-16 17:51, Yuanhan Liu: > > On Wed, Nov 30, 2016 at 02:54:14PM +, Ferruh Yigit wrote: > > > On 11/21/2016 10:43 PM, Thomas Monjalon wrote: > > > > Add a check for commits fixing a released bug. > > > > Such commits are fo

Re: [dpdk-dev] [PATCH] scripts: check cc stable mailing list in commit

2017-01-16 Thread Yuanhan Liu
On Mon, Jan 16, 2017 at 10:38:02AM +, Ferruh Yigit wrote: > On 1/16/2017 9:51 AM, Yuanhan Liu wrote: > > On Wed, Nov 30, 2016 at 02:54:14PM +, Ferruh Yigit wrote: > >> On 11/21/2016 10:43 PM, Thomas Monjalon wrote: > >>> Add a check for commits fixing a released bug. > >>> Such commits are

Re: [dpdk-dev] [PATCH] scripts: check cc stable mailing list in commit

2017-01-16 Thread Thomas Monjalon
2017-01-16 17:51, Yuanhan Liu: > On Wed, Nov 30, 2016 at 02:54:14PM +, Ferruh Yigit wrote: > > On 11/21/2016 10:43 PM, Thomas Monjalon wrote: > > > Add a check for commits fixing a released bug. > > > Such commits are found thanks to scripts/git-log-fixes.sh. > > > They must be sent CC: sta...@

Re: [dpdk-dev] [PATCH] scripts: check cc stable mailing list in commit

2017-01-16 Thread Ferruh Yigit
On 1/16/2017 9:51 AM, Yuanhan Liu wrote: > On Wed, Nov 30, 2016 at 02:54:14PM +, Ferruh Yigit wrote: >> On 11/21/2016 10:43 PM, Thomas Monjalon wrote: >>> Add a check for commits fixing a released bug. >>> Such commits are found thanks to scripts/git-log-fixes.sh. >>> They must be sent CC: sta.

Re: [dpdk-dev] [PATCH] scripts: check cc stable mailing list in commit

2017-01-16 Thread Yuanhan Liu
On Wed, Nov 30, 2016 at 02:54:14PM +, Ferruh Yigit wrote: > On 11/21/2016 10:43 PM, Thomas Monjalon wrote: > > Add a check for commits fixing a released bug. > > Such commits are found thanks to scripts/git-log-fixes.sh. > > They must be sent CC: sta...@dpdk.org. > > In order to avoid forgettin

[dpdk-dev] [PATCH] scripts: check cc stable mailing list in commit

2016-11-30 Thread Thomas Monjalon
2016-11-30 15:26, Bruce Richardson: > On Wed, Nov 30, 2016 at 04:09:47PM +0100, Thomas Monjalon wrote: > > 2016-11-30 14:54, Ferruh Yigit: > > > On 11/21/2016 10:43 PM, Thomas Monjalon wrote: > > > > +stablefixes=$($selfdir/git-log-fixes.sh $range | sed '/(N\/A)$/d' | > > > > cut -d' ' -f2) > > >

[dpdk-dev] [PATCH] scripts: check cc stable mailing list in commit

2016-11-30 Thread Thomas Monjalon
2016-11-30 14:54, Ferruh Yigit: > On 11/21/2016 10:43 PM, Thomas Monjalon wrote: > > +stablefixes=$($selfdir/git-log-fixes.sh $range | sed '/(N\/A)$/d' | cut > > -d' ' -f2) > > This breaks the "check-git-log.sh -N" usage, since "-N" is not a valid > range for git-log-fixes.sh. > Generates warnin

[dpdk-dev] [PATCH] scripts: check cc stable mailing list in commit

2016-11-30 Thread Bruce Richardson
On Wed, Nov 30, 2016 at 04:31:46PM +0100, Thomas Monjalon wrote: > 2016-11-30 15:26, Bruce Richardson: > > On Wed, Nov 30, 2016 at 04:09:47PM +0100, Thomas Monjalon wrote: > > > 2016-11-30 14:54, Ferruh Yigit: > > > > On 11/21/2016 10:43 PM, Thomas Monjalon wrote: > > > > > +stablefixes=$($selfdir/

[dpdk-dev] [PATCH] scripts: check cc stable mailing list in commit

2016-11-30 Thread Bruce Richardson
On Wed, Nov 30, 2016 at 04:09:47PM +0100, Thomas Monjalon wrote: > 2016-11-30 14:54, Ferruh Yigit: > > On 11/21/2016 10:43 PM, Thomas Monjalon wrote: > > > +stablefixes=$($selfdir/git-log-fixes.sh $range | sed '/(N\/A)$/d' | cut > > > -d' ' -f2) > > > > This breaks the "check-git-log.sh -N" usag

[dpdk-dev] [PATCH] scripts: check cc stable mailing list in commit

2016-11-30 Thread Ferruh Yigit
On 11/21/2016 10:43 PM, Thomas Monjalon wrote: > Add a check for commits fixing a released bug. > Such commits are found thanks to scripts/git-log-fixes.sh. > They must be sent CC: stable at dpdk.org. > In order to avoid forgetting CC, this mail header can be written > in the git commit message. >

[dpdk-dev] [PATCH] scripts: check cc stable mailing list in commit

2016-11-21 Thread Thomas Monjalon
Add a check for commits fixing a released bug. Such commits are found thanks to scripts/git-log-fixes.sh. They must be sent CC: stable at dpdk.org. In order to avoid forgetting CC, this mail header can be written in the git commit message. Signed-off-by: Thomas Monjalon --- scripts/check-git-log