Re: [ovs-dev] [PATCH] checkpatch: Ignore utitilies/bugtool.

2019-09-17 Thread Ben Pfaff
On Mon, Sep 16, 2019 at 09:35:48AM -0700, Gregory Rose wrote:
> On 9/12/2019 11:11 AM, William Tu wrote:
> > Signed-off-by: William Tu 
> > ---
> >   utilities/checkpatch.py | 2 ++
> >   1 file changed, 2 insertions(+)
> > 
> > diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py
> > index f8fa00e306a8..a9f27b52f3c8 100755
> > --- a/utilities/checkpatch.py
> > +++ b/utilities/checkpatch.py
> > @@ -844,6 +844,8 @@ def ovs_checkpatch_parse(text, filename, author=None, 
> > committer=None):
> >   # for a common style.
> >   if current_file.startswith('include/sparse'):
> >   continue
> > +if current_file.startswith('utilities/bugtool'):
> > +continue
> >   run_checks(current_file, cmp_line, lineno)
> >   run_file_checks(text)
> 
> Seems fine to me.
> 
> Reviewed-by: Greg Rose 

Thanks, William and Greg.  I applied this to master.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH] checkpatch: Ignore utitilies/bugtool.

2019-09-16 Thread Gregory Rose

On 9/12/2019 11:11 AM, William Tu wrote:

Signed-off-by: William Tu 
---
  utilities/checkpatch.py | 2 ++
  1 file changed, 2 insertions(+)

diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py
index f8fa00e306a8..a9f27b52f3c8 100755
--- a/utilities/checkpatch.py
+++ b/utilities/checkpatch.py
@@ -844,6 +844,8 @@ def ovs_checkpatch_parse(text, filename, author=None, 
committer=None):
  # for a common style.
  if current_file.startswith('include/sparse'):
  continue
+if current_file.startswith('utilities/bugtool'):
+continue
  run_checks(current_file, cmp_line, lineno)
  
  run_file_checks(text)


Seems fine to me.

Reviewed-by: Greg Rose 
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev