summaryrefslogtreecommitdiff
path: root/utilities
diff options
context:
space:
mode:
authorIlya Maximets <i.maximets@samsung.com>2017-07-14 13:57:23 +0300
committerRussell Bryant <russell@ovn.org>2017-07-26 16:42:11 -0400
commitb90cfa86a2f6749be46cd7920d3e8f72b972c1ce (patch)
treebdaf822d85583cf1af75849aca52c89f2bb12bbb /utilities
parent81c2316f71784ee604805ddac52921e5b5d8461f (diff)
downloadopenvswitch-b90cfa86a2f6749be46cd7920d3e8f72b972c1ce.tar.gz
checkpatch: Allow checking more than one file.
Currently to check more than one patch or file it's required to invoke script for each file separately. Fix that by iterating over all the passed filenames. Note: If '-f' option passed, all the files treated as usual files. Without '-f' all the files treated as patch files. Signed-off-by: Ilya Maximets <i.maximets@samsung.com> Acked-by: Aaron Conole <aconole@redhat.com> Signed-off-by: Russell Bryant <russell@ovn.org>
Diffstat (limited to 'utilities')
-rwxr-xr-xutilities/checkpatch.py15
1 files changed, 10 insertions, 5 deletions
diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py
index 4a92890fb..7ccec51df 100755
--- a/utilities/checkpatch.py
+++ b/utilities/checkpatch.py
@@ -408,7 +408,7 @@ def usage():
Open vSwitch checkpatch.py
Checks a patch for trivial mistakes.
usage:
-%s [options] [PATCH | -f SOURCE | -1 | -2 | ...]
+%s [options] [PATCH1 [PATCH2 ...] | -f SOURCE1 [SOURCE2 ...] | -1 | -2 | ...]
Input options:
-f|--check-file Arguments are source files, not patches.
@@ -513,13 +513,18 @@ if __name__ == '__main__':
status = -1
sys.exit(status)
- try:
- filename = args[0]
- except:
+ if not args:
if sys.stdin.isatty():
usage()
sys.exit(-1)
result = ovs_checkpatch_parse(sys.stdin.read(), '-')
ovs_checkpatch_print_result(result)
sys.exit(result)
- sys.exit(ovs_checkpatch_file(filename))
+
+ status = 0
+ for filename in args:
+ print('== Checking "%s" ==' % filename)
+ result = ovs_checkpatch_file(filename)
+ if result:
+ status = -1
+ sys.exit(status)