mirror of
https://github.com/openvswitch/ovs
synced 2025-08-30 22:05:19 +00:00
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>
This commit is contained in:
committed by
Russell Bryant
parent
81c2316f71
commit
b90cfa86a2
@@ -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)
|
||||
|
Reference in New Issue
Block a user