[dpdk-dev] [PATCH] scripts: check commits with checkpatch

2016-06-07 Thread Thomas Monjalon
2016-06-02 13:33, Bruce Richardson: > On Thu, Jun 02, 2016 at 11:13:15AM +0200, Thomas Monjalon wrote: > > The new option -n allows to give a number of commits to check > > from the git HEAD. > > If neither -n nor patch files are given, the commits after > > origin/master are checked. > > > >

[dpdk-dev] [PATCH] scripts: check commits with checkpatch

2016-06-02 Thread Thomas Monjalon
2016-06-02 13:33, Bruce Richardson: > Testing this out here, I find that git format-patch includes the diff stats > in the output, which then can trigger long-line warnings for the commit > message. > > WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description > (prefer a

[dpdk-dev] [PATCH] scripts: check commits with checkpatch

2016-06-02 Thread Bruce Richardson
On Thu, Jun 02, 2016 at 03:15:54PM +0200, Thomas Monjalon wrote: > 2016-06-02 13:33, Bruce Richardson: > > Testing this out here, I find that git format-patch includes the diff stats > > in the output, which then can trigger long-line warnings for the commit > > message. > > > >

[dpdk-dev] [PATCH] scripts: check commits with checkpatch

2016-06-02 Thread Bruce Richardson
On Thu, Jun 02, 2016 at 11:13:15AM +0200, Thomas Monjalon wrote: > The new option -n allows to give a number of commits to check > from the git HEAD. > If neither -n nor patch files are given, the commits after > origin/master are checked. > Yep, I love it, exactly what I wanted to replace my

[dpdk-dev] [PATCH] scripts: check commits with checkpatch

2016-06-02 Thread Thomas Monjalon
The new option -n allows to give a number of commits to check from the git HEAD. If neither -n nor patch files are given, the commits after origin/master are checked. Signed-off-by: Thomas Monjalon --- scripts/checkpatches.sh | 45 - 1 file changed,