Message ID | 1568311877-14135-1-git-send-email-u9012063@gmail.com |
---|---|
State | Accepted |
Headers | show |
Series | [ovs-dev] checkpatch: Ignore utitilies/bugtool. | expand |
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>
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 --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)
Signed-off-by: William Tu <u9012063@gmail.com> --- utilities/checkpatch.py | 2 ++ 1 file changed, 2 insertions(+)