diff options
author | Ilya Maximets <i.maximets@samsung.com> | 2017-07-14 13:57:23 +0300 |
---|---|---|
committer | Russell Bryant <russell@ovn.org> | 2017-07-26 16:42:11 -0400 |
commit | b90cfa86a2f6749be46cd7920d3e8f72b972c1ce (patch) | |
tree | bdaf822d85583cf1af75849aca52c89f2bb12bbb /utilities | |
parent | 81c2316f71784ee604805ddac52921e5b5d8461f (diff) | |
download | openvswitch-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-x | utilities/checkpatch.py | 15 |
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) |