[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH v2 00/10] python: futurize --stage1
From: |
no-reply |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH v2 00/10] python: futurize --stage1 (Python 3 compatibility) |
Date: |
Fri, 8 Jun 2018 05:48:22 -0700 (PDT) |
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: address@hidden
Subject: [Qemu-devel] [PATCH v2 00/10] python: futurize --stage1 (Python 3
compatibility)
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BASE=base
n=1
total=$(git log --oneline $BASE.. | wc -l)
failed=0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
commits="$(git log --format=%H --reverse $BASE..)"
for c in $commits; do
echo "Checking PATCH $n/$total: $(git log -n 1 --format=%s $c)..."
if ! git show $c --format=email | ./scripts/checkpatch.pl --mailback -; then
failed=1
echo
fi
n=$((n+1))
done
exit $failed
=== TEST SCRIPT END ===
Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
From https://github.com/patchew-project/qemu
* [new tag] patchew/address@hidden -> patchew/address@hidden
Switched to a new branch 'test'
034a17b732 python: futurize -f lib2to3.fixes.fix_numliterals
24815c6337 python: futurize -f lib2to3.fixes.fix_except
84c2eb4a4a python: futurize -f lib2to3.fixes.fix_renames
8e21c5b4fd python: futurize -f lib2to3.fixes.fix_tuple_params
8314382c8c python: futurize -f lib2to3.fixes.fix_reduce
db6bd54016 python: futurize -f lib2to3.fixes.fix_standarderror
26518cce56 python: futurize -f lib2to3.fixes.fix_has_key
710ff909f0 python: futurize -f libfuturize.fixes.fix_next_call
332ca44612 python: futurize -f libfuturize.fixes.fix_absolute_import
ebdf9a3fe6 python: futurize -f libfuturize.fixes.fix_print_with_import
=== OUTPUT BEGIN ===
Checking PATCH 1/10: python: futurize -f
libfuturize.fixes.fix_print_with_import...
ERROR: line over 90 characters
#42: FILE: scripts/analyse-9p-simpletrace.py:86:
+ print("RERROR (tag =", tag, ", id =", symbol_9p[id], ", err =
\"", os.strerror(err), "\")")
ERROR: line over 90 characters
#46: FILE: scripts/analyse-9p-simpletrace.py:89:
+ print("TVERSION (tag =", tag, ", msize =", msize, ", version
=", version, ")")
ERROR: line over 90 characters
#50: FILE: scripts/analyse-9p-simpletrace.py:92:
+ print("RVERSION (tag =", tag, ", msize =", msize, ", version
=", version, ")")
ERROR: line over 90 characters
#54: FILE: scripts/analyse-9p-simpletrace.py:95:
+ print("TATTACH (tag =", tag, ", fid =", fid, ", afid =", afid,
", uname =", uname, ", aname =", aname, ")")
ERROR: line over 90 characters
#58: FILE: scripts/analyse-9p-simpletrace.py:98:
+ print("RATTACH (tag =", tag, ", qid={type =", type, ", version
=", version, ", path =", path, "})")
ERROR: line over 90 characters
#66: FILE: scripts/analyse-9p-simpletrace.py:104:
+ print("RSTAT (tag =", tag, ", mode =", mode, ", atime =",
atime, ", mtime =", mtime, ", length =", length, ")")
ERROR: line over 90 characters
#70: FILE: scripts/analyse-9p-simpletrace.py:107:
+ print("TGETATTR (tag =", tag, ", fid =", fid, ", request_mask
=", hex(request_mask), ")")
ERROR: line over 90 characters
#74: FILE: scripts/analyse-9p-simpletrace.py:110:
+ print("RGETATTR (tag =", tag, ", result_mask =",
hex(result_mask), ", mode =", oct(mode), ", uid =", uid, ", gid =", gid, ")")
ERROR: line over 90 characters
#78: FILE: scripts/analyse-9p-simpletrace.py:113:
+ print("TWALK (tag =", tag, ", fid =", fid, ", newfid =",
newfid, ", nwnames =", nwnames, ")")
ERROR: line over 90 characters
#82: FILE: scripts/analyse-9p-simpletrace.py:116:
+ print("RWALK (tag =", tag, ", nwnames =", nwnames, ", qids =",
hex(qids), ")")
WARNING: line over 80 characters
#86: FILE: scripts/analyse-9p-simpletrace.py:119:
+ print("TOPEN (tag =", tag, ", fid =", fid, ", mode =",
oct(mode), ")")
ERROR: line over 90 characters
#90: FILE: scripts/analyse-9p-simpletrace.py:122:
+ print("ROPEN (tag =", tag, ", qid={type =", type, ", version
=", version, ", path =", path, "}, iounit =", iounit, ")")
ERROR: line over 90 characters
#94: FILE: scripts/analyse-9p-simpletrace.py:125:
+ print("TLCREATE (tag =", tag, ", dfid =", dfid, ", flags =",
oct(flags), ", mode =", oct(mode), ", gid =", gid, ")")
ERROR: line over 90 characters
#98: FILE: scripts/analyse-9p-simpletrace.py:128:
+ print("RLCREATE (tag =", tag, ", qid={type =", type, ",
version =", version, ", path =", path, "}, iounit =", iounit, ")")
WARNING: line over 80 characters
#102: FILE: scripts/analyse-9p-simpletrace.py:131:
+ print("TFSYNC (tag =", tag, ", fid =", fid, ", datasync =",
datasync, ")")
ERROR: line over 90 characters
#110: FILE: scripts/analyse-9p-simpletrace.py:137:
+ print("TREAD (tag =", tag, ", fid =", fid, ", off =", off, ",
max_count =", max_count, ")")
WARNING: line over 80 characters
#114: FILE: scripts/analyse-9p-simpletrace.py:140:
+ print("RREAD (tag =", tag, ", count =", count, ", err =", err,
")")
ERROR: line over 90 characters
#118: FILE: scripts/analyse-9p-simpletrace.py:143:
+ print("TREADDIR (tag =", tag, ", fid =", fid, ", offset =",
offset, ", max_count =", max_count, ")")
ERROR: line over 90 characters
#122: FILE: scripts/analyse-9p-simpletrace.py:146:
+ print("RREADDIR (tag =", tag, ", count =", count, ", retval
=", retval, ")")
ERROR: line over 90 characters
#126: FILE: scripts/analyse-9p-simpletrace.py:149:
+ print("TWRITE (tag =", tag, ", fid =", fid, ", off =", off, ",
count =", count, ", cnt =", cnt, ")")
WARNING: line over 80 characters
#130: FILE: scripts/analyse-9p-simpletrace.py:152:
+ print("RWRITE (tag =", tag, ", total =", total, ", err =",
err, ")")
ERROR: line over 90 characters
#134: FILE: scripts/analyse-9p-simpletrace.py:155:
+ print("TCREATE (tag =", tag, ", fid =", fid, ", perm =",
oct(perm), ", name =", name, ", mode =", oct(mode), ")")
ERROR: line over 90 characters
#138: FILE: scripts/analyse-9p-simpletrace.py:158:
+ print("RCREATE (tag =", tag, ", qid={type =", type, ",
version =", version, ", path =", path, "}, iounit =", iounit, ")")
ERROR: line over 90 characters
#142: FILE: scripts/analyse-9p-simpletrace.py:161:
+ print("TSYMLINK (tag =", tag, ", fid =", fid, ", name =",
name, ", symname =", symname, ", gid =", gid, ")")
ERROR: line over 90 characters
#146: FILE: scripts/analyse-9p-simpletrace.py:164:
+ print("RSYMLINK (tag =", tag, ", qid={type =", type, ",
version =", version, ", path =", path, "})")
ERROR: line over 90 characters
#154: FILE: scripts/analyse-9p-simpletrace.py:170:
+ print("TLINK (tag =", tag, ", dfid =", dfid, ", oldfid =",
oldfid, ", name =", name, ")")
ERROR: line over 90 characters
#162: FILE: scripts/analyse-9p-simpletrace.py:176:
+ print("TWSTAT (tag =", tag, ", fid =", fid, ", mode =",
oct(mode), ", atime =", atime, "mtime =", mtime, ")")
ERROR: line over 90 characters
#166: FILE: scripts/analyse-9p-simpletrace.py:179:
+ print("TMKNOD (tag =", tag, ", fid =", fid, ", mode =",
oct(mode), ", major =", major, ", minor =", minor, ")")
ERROR: line over 90 characters
#170: FILE: scripts/analyse-9p-simpletrace.py:182:
+ print("TLOCK (tag =", tag, ", fid =", fid, "type =", type, ",
start =", start, ", length =", length, ")")
ERROR: line over 90 characters
#178: FILE: scripts/analyse-9p-simpletrace.py:188:
+ print("TGETLOCK (tag =", tag, ", fid =", fid, "type =", type,
", start =", start, ", length =", length, ")")
ERROR: line over 90 characters
#182: FILE: scripts/analyse-9p-simpletrace.py:191:
+ print("RGETLOCK (tag =", tag, "type =", type, ", start =",
start, ", length =", length, ", proc_id =", proc_id, ")")
ERROR: line over 90 characters
#186: FILE: scripts/analyse-9p-simpletrace.py:194:
+ print("TMKDIR (tag =", tag, ", fid =", fid, ", name =", name,
", mode =", mode, ", gid =", gid, ")")
ERROR: line over 90 characters
#190: FILE: scripts/analyse-9p-simpletrace.py:197:
+ print("RMKDIR (tag =", tag, ", qid={type =", type, ", version
=", version, ", path =", path, "}, err =", err, ")")
ERROR: line over 90 characters
#194: FILE: scripts/analyse-9p-simpletrace.py:200:
+ print("TXATTRWALK (tag =", tag, ", fid =", fid, ", newfid =",
newfid, ", xattr name =", name, ")")
ERROR: line over 90 characters
#202: FILE: scripts/analyse-9p-simpletrace.py:206:
+ print("TXATTRCREATE (tag =", tag, ", fid =", fid, ", name =",
name, ", xattrsize =", size, ", flags =", flags, ")")
WARNING: line over 80 characters
#845: FILE: scripts/vmstate-static-checker.py:182:
+ print("expected field \"" + s_item["field"] + "\",", end='
')
WARNING: line over 80 characters
#901: FILE: scripts/vmstate-static-checker.py:293:
+ print("Section \"" + sec + "\", Description \"" + desc + "\":",
end=' ')
WARNING: line over 80 characters
#1009: FILE: scripts/vmstate-static-checker.py:419:
+ print("Section \"" + sec + "\": Entry \"" + entry + "\"",
end=' ')
WARNING: line over 80 characters
#1129: FILE: tests/image-fuzzer/runner.py:41:
+ "'--config' and '--command' options are not supported.",
file=sys.stderr)
WARNING: line over 80 characters
#1140: FILE: tests/image-fuzzer/runner.py:161:
+ print("Error: The working directory '%s' cannot be used. Reason:
%s"\
ERROR: line over 90 characters
#1169: FILE: tests/image-fuzzer/runner.py:359:
+ print("Error: %s\n\nTry 'runner.py --help' for more information" % e,
file=sys.stderr)
WARNING: line over 80 characters
#1231: FILE: tests/migration/guestperf/engine.py:121:
+ print("Sleeping %d seconds for initial guest workload run" %
self._sleep)
ERROR: line over 90 characters
#1249: FILE: tests/migration/guestperf/engine.py:220:
+ print("Sleeping %d seconds for final guest workload
run" % self._sleep)
WARNING: line over 80 characters
#1258: FILE: tests/migration/guestperf/engine.py:231:
+ print("Iter %d: remain %5dMB of %5dMB (total %5dMB @
%5dMb/sec)" % (
ERROR: line over 90 characters
#1270: FILE: tests/migration/guestperf/engine.py:241:
+ print("No completion after %d iterations over RAM" %
scenario._max_iters)
ERROR: line over 90 characters
#1286: FILE: tests/migration/guestperf/engine.py:255:
+ print("Switching to post-copy after %d iterations" %
scenario._post_copy_iters)
WARNING: line over 80 characters
#1295: FILE: tests/migration/guestperf/engine.py:263:
+ print("Pausing VM after %d iterations" %
scenario._pause_iters)
ERROR: line over 90 characters
#1653: FILE: tests/qemu-iotests/qcow2.py:226:
+ print("'%s' is not a valid group, try 'incompatible', 'compatible', or
'autoclear'" % group)
total: 36 errors, 12 warnings, 1481 lines checked
Your patch has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
Checking PATCH 2/10: python: futurize -f
libfuturize.fixes.fix_absolute_import...
Checking PATCH 3/10: python: futurize -f libfuturize.fixes.fix_next_call...
Checking PATCH 4/10: python: futurize -f lib2to3.fixes.fix_has_key...
Checking PATCH 5/10: python: futurize -f lib2to3.fixes.fix_standarderror...
Checking PATCH 6/10: python: futurize -f lib2to3.fixes.fix_reduce...
Checking PATCH 7/10: python: futurize -f lib2to3.fixes.fix_tuple_params...
Checking PATCH 8/10: python: futurize -f lib2to3.fixes.fix_renames...
Checking PATCH 9/10: python: futurize -f lib2to3.fixes.fix_except...
Checking PATCH 10/10: python: futurize -f lib2to3.fixes.fix_numliterals...
=== OUTPUT END ===
Test command exited with code: 1
---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to address@hidden
- [Qemu-block] [PATCH v2 02/10] python: futurize -f libfuturize.fixes.fix_absolute_import, (continued)
- [Qemu-block] [PATCH v2 02/10] python: futurize -f libfuturize.fixes.fix_absolute_import, Eduardo Habkost, 2018/06/08
- [Qemu-block] [PATCH v2 04/10] python: futurize -f lib2to3.fixes.fix_has_key, Eduardo Habkost, 2018/06/08
- [Qemu-block] [PATCH v2 05/10] python: futurize -f lib2to3.fixes.fix_standarderror, Eduardo Habkost, 2018/06/08
- [Qemu-block] [PATCH v2 06/10] python: futurize -f lib2to3.fixes.fix_reduce, Eduardo Habkost, 2018/06/08
- [Qemu-block] [PATCH v2 07/10] python: futurize -f lib2to3.fixes.fix_tuple_params, Eduardo Habkost, 2018/06/08
- [Qemu-block] [PATCH v2 08/10] python: futurize -f lib2to3.fixes.fix_renames, Eduardo Habkost, 2018/06/08
- [Qemu-block] [PATCH v2 09/10] python: futurize -f lib2to3.fixes.fix_except, Eduardo Habkost, 2018/06/08
- [Qemu-block] [PATCH v2 10/10] python: futurize -f lib2to3.fixes.fix_numliterals, Eduardo Habkost, 2018/06/08
- Re: [Qemu-block] [Qemu-devel] [PATCH v2 00/10] python: futurize --stage1 (Python 3 compatibility), no-reply, 2018/06/08
- Re: [Qemu-block] [Qemu-devel] [PATCH v2 00/10] python: futurize --stage1 (Python 3 compatibility),
no-reply <=