[ovs-dev] [PATCH 3/4] checkpatch: Allow checking more than one file.
Ilya Maximets
i.maximets at samsung.com
Fri Jul 14 10:57:23 UTC 2017
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 at samsung.com>
---
utilities/checkpatch.py | 15 ++++++++++-----
1 file changed, 10 insertions(+), 5 deletions(-)
diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py
index 4a92890..7ccec51 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)
--
2.7.4
More information about the dev
mailing list