[dpdk-dev] [PATCH 3/3] scripts: check headline of drivers commits

2016-07-06 Thread Thomas Monjalon
2016-07-06 13:59, N?lio Laranjeiro: > > +# check headline prefix when touching only drivers/, e.g. net/ > > +bad=$(for commit in $commits ; do > > + headline=$(git log --format='%s' -1 $commit) > > + files=$(git diff-tree --no-commit-id --name-only -r $commit) > > + [ -z "$(echo "$files" | gr

[dpdk-dev] [PATCH 3/3] scripts: check headline of drivers commits

2016-07-06 Thread Thomas Monjalon
2016-07-06 12:09, Bruce Richardson: > On Wed, Jul 06, 2016 at 12:38:50PM +0200, Thomas Monjalon wrote: > > From: Nelio Laranjeiro > > +# check headline prefix when touching only drivers/, e.g. net/ > > +bad=$(for commit in $commits ; do > > + headline=$(git log --format='%s' -1 $commit) > > +

[dpdk-dev] [PATCH 3/3] scripts: check headline of drivers commits

2016-07-06 Thread Nélio Laranjeiro
Thomas, On Wed, Jul 06, 2016 at 12:38:50PM +0200, Thomas Monjalon wrote: > From: Nelio Laranjeiro > > A driver patch under net should start with "net/" or if > a patch touch multiple drivers, it should only start with "net:". > The same apply for crypto. > A patch touching all drivers (net + cry

[dpdk-dev] [PATCH 3/3] scripts: check headline of drivers commits

2016-07-06 Thread Thomas Monjalon
From: Nelio Laranjeiro A driver patch under net should start with "net/" or if a patch touch multiple drivers, it should only start with "net:". The same apply for crypto. A patch touching all drivers (net + crypto) should start with "drivers:". Longer prefixes like "net/mlx:" (for mlx4/mlx5) or

[dpdk-dev] [PATCH 3/3] scripts: check headline of drivers commits

2016-07-06 Thread Bruce Richardson
On Wed, Jul 06, 2016 at 12:38:50PM +0200, Thomas Monjalon wrote: > From: Nelio Laranjeiro > > A driver patch under net should start with "net/" or if > a patch touch multiple drivers, it should only start with "net:". > The same apply for crypto. > A patch touching all drivers (net + crypto) shou