diff mbox series

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

Message ID 1568311877-14135-1-git-send-email-u9012063@gmail.com
State Accepted
Headers show
Series [ovs-dev] checkpatch: Ignore utitilies/bugtool. | expand

Commit Message

William Tu Sept. 12, 2019, 6:11 p.m. UTC 
Signed-off-by: William Tu <u9012063@gmail.com>
---
 utilities/checkpatch.py | 2 ++
 1 file changed, 2 insertions(+)

Comments

Gregory Rose Sept. 16, 2019, 4:35 p.m. UTC | #1 
On 9/12/2019 11:11 AM, William Tu wrote:
> Signed-off-by: William Tu <u9012063@gmail.com>
> ---
>   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 <gvrose8192@gmail.com>
Ben Pfaff Sept. 17, 2019, 9:41 p.m. UTC | #2 
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 <u9012063@gmail.com>
> > ---
> >   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 <gvrose8192@gmail.com>

Thanks, William and Greg.  I applied this to master.
diff mbox series

Patch

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)