[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC 00/10] [TESTING NEEDED] python: futurize --stage1
From: |
no-reply |
Subject: |
Re: [Qemu-devel] [RFC 00/10] [TESTING NEEDED] python: futurize --stage1 (Python 3 compatibility) |
Date: |
Fri, 11 May 2018 15:34:33 -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] [RFC 00/10] [TESTING NEEDED] 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
t [tag update] patchew/address@hidden -> patchew/address@hidden
* [new tag] patchew/address@hidden -> patchew/address@hidden
Switched to a new branch 'test'
29e77f7e45 python: futurize -f lib2to3.fixes.fix_numliterals
06044c1048 python: futurize -f lib2to3.fixes.fix_except
aecdedee26 python: futurize -f lib2to3.fixes.fix_renames
2be7292f53 python: futurize -f lib2to3.fixes.fix_tuple_params
0b3e5f083c python: futurize -f lib2to3.fixes.fix_reduce
6d7a39f3c6 python: futurize -f lib2to3.fixes.fix_standarderror
0c8beeea09 python: futurize -f lib2to3.fixes.fix_has_key
4f0075d69d python: futurize -f libfuturize.fixes.fix_next_call
66698bae73 python: futurize -f libfuturize.fixes.fix_absolute_import
b48c4f449d 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
#40: FILE: scripts/analyse-9p-simpletrace.py:86:
+ print("RERROR (tag =", tag, ", id =", symbol_9p[id], ", err =
\"", os.strerror(err), "\")")
ERROR: line over 90 characters
#44: FILE: scripts/analyse-9p-simpletrace.py:89:
+ print("TVERSION (tag =", tag, ", msize =", msize, ", version
=", version, ")")
ERROR: line over 90 characters
#48: FILE: scripts/analyse-9p-simpletrace.py:92:
+ print("RVERSION (tag =", tag, ", msize =", msize, ", version
=", version, ")")
ERROR: line over 90 characters
#52: FILE: scripts/analyse-9p-simpletrace.py:95:
+ print("TATTACH (tag =", tag, ", fid =", fid, ", afid =", afid,
", uname =", uname, ", aname =", aname, ")")
ERROR: line over 90 characters
#56: FILE: scripts/analyse-9p-simpletrace.py:98:
+ print("RATTACH (tag =", tag, ", qid={type =", type, ", version
=", version, ", path =", path, "})")
ERROR: line over 90 characters
#64: FILE: scripts/analyse-9p-simpletrace.py:104:
+ print("RSTAT (tag =", tag, ", mode =", mode, ", atime =",
atime, ", mtime =", mtime, ", length =", length, ")")
ERROR: line over 90 characters
#68: FILE: scripts/analyse-9p-simpletrace.py:107:
+ print("TGETATTR (tag =", tag, ", fid =", fid, ", request_mask
=", hex(request_mask), ")")
ERROR: line over 90 characters
#72: 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
#76: FILE: scripts/analyse-9p-simpletrace.py:113:
+ print("TWALK (tag =", tag, ", fid =", fid, ", newfid =",
newfid, ", nwnames =", nwnames, ")")
ERROR: line over 90 characters
#80: FILE: scripts/analyse-9p-simpletrace.py:116:
+ print("RWALK (tag =", tag, ", nwnames =", nwnames, ", qids =",
hex(qids), ")")
WARNING: line over 80 characters
#84: FILE: scripts/analyse-9p-simpletrace.py:119:
+ print("TOPEN (tag =", tag, ", fid =", fid, ", mode =",
oct(mode), ")")
ERROR: line over 90 characters
#88: 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
#92: 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
#96: 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
#100: FILE: scripts/analyse-9p-simpletrace.py:131:
+ print("TFSYNC (tag =", tag, ", fid =", fid, ", datasync =",
datasync, ")")
ERROR: line over 90 characters
#108: FILE: scripts/analyse-9p-simpletrace.py:137:
+ print("TREAD (tag =", tag, ", fid =", fid, ", off =", off, ",
max_count =", max_count, ")")
WARNING: line over 80 characters
#112: FILE: scripts/analyse-9p-simpletrace.py:140:
+ print("RREAD (tag =", tag, ", count =", count, ", err =", err,
")")
ERROR: line over 90 characters
#116: FILE: scripts/analyse-9p-simpletrace.py:143:
+ print("TREADDIR (tag =", tag, ", fid =", fid, ", offset =",
offset, ", max_count =", max_count, ")")
ERROR: line over 90 characters
#120: FILE: scripts/analyse-9p-simpletrace.py:146:
+ print("RREADDIR (tag =", tag, ", count =", count, ", retval
=", retval, ")")
ERROR: line over 90 characters
#124: FILE: scripts/analyse-9p-simpletrace.py:149:
+ print("TWRITE (tag =", tag, ", fid =", fid, ", off =", off, ",
count =", count, ", cnt =", cnt, ")")
WARNING: line over 80 characters
#128: FILE: scripts/analyse-9p-simpletrace.py:152:
+ print("RWRITE (tag =", tag, ", total =", total, ", err =",
err, ")")
ERROR: line over 90 characters
#132: 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
#136: 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
#140: FILE: scripts/analyse-9p-simpletrace.py:161:
+ print("TSYMLINK (tag =", tag, ", fid =", fid, ", name =",
name, ", symname =", symname, ", gid =", gid, ")")
ERROR: line over 90 characters
#144: FILE: scripts/analyse-9p-simpletrace.py:164:
+ print("RSYMLINK (tag =", tag, ", qid={type =", type, ",
version =", version, ", path =", path, "})")
ERROR: line over 90 characters
#152: FILE: scripts/analyse-9p-simpletrace.py:170:
+ print("TLINK (tag =", tag, ", dfid =", dfid, ", oldfid =",
oldfid, ", name =", name, ")")
ERROR: line over 90 characters
#160: 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
#164: 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
#168: FILE: scripts/analyse-9p-simpletrace.py:182:
+ print("TLOCK (tag =", tag, ", fid =", fid, "type =", type, ",
start =", start, ", length =", length, ")")
ERROR: line over 90 characters
#176: FILE: scripts/analyse-9p-simpletrace.py:188:
+ print("TGETLOCK (tag =", tag, ", fid =", fid, "type =", type,
", start =", start, ", length =", length, ")")
ERROR: line over 90 characters
#180: 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
#184: FILE: scripts/analyse-9p-simpletrace.py:194:
+ print("TMKDIR (tag =", tag, ", fid =", fid, ", name =", name,
", mode =", mode, ", gid =", gid, ")")
ERROR: line over 90 characters
#188: 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
#192: FILE: scripts/analyse-9p-simpletrace.py:200:
+ print("TXATTRWALK (tag =", tag, ", fid =", fid, ", newfid =",
newfid, ", xattr name =", name, ")")
ERROR: line over 90 characters
#200: FILE: scripts/analyse-9p-simpletrace.py:206:
+ print("TXATTRCREATE (tag =", tag, ", fid =", fid, ", name =",
name, ", xattrsize =", size, ", flags =", flags, ")")
WARNING: line over 80 characters
#843: FILE: scripts/vmstate-static-checker.py:182:
+ print("expected field \"" + s_item["field"] + "\",", end='
')
WARNING: line over 80 characters
#899: FILE: scripts/vmstate-static-checker.py:293:
+ print("Section \"" + sec + "\", Description \"" + desc + "\":",
end=' ')
WARNING: line over 80 characters
#1007: FILE: scripts/vmstate-static-checker.py:419:
+ print("Section \"" + sec + "\": Entry \"" + entry + "\"",
end=' ')
WARNING: line over 80 characters
#1127: FILE: tests/image-fuzzer/runner.py:41:
+ "'--config' and '--command' options are not supported.",
file=sys.stderr)
WARNING: line over 80 characters
#1138: FILE: tests/image-fuzzer/runner.py:161:
+ print("Error: The working directory '%s' cannot be used. Reason:
%s"\
ERROR: line over 90 characters
#1167: 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
#1229: FILE: tests/migration/guestperf/engine.py:121:
+ print("Sleeping %d seconds for initial guest workload run" %
self._sleep)
ERROR: line over 90 characters
#1247: FILE: tests/migration/guestperf/engine.py:220:
+ print("Sleeping %d seconds for final guest workload
run" % self._sleep)
WARNING: line over 80 characters
#1256: FILE: tests/migration/guestperf/engine.py:231:
+ print("Iter %d: remain %5dMB of %5dMB (total %5dMB @
%5dMb/sec)" % (
ERROR: line over 90 characters
#1268: FILE: tests/migration/guestperf/engine.py:241:
+ print("No completion after %d iterations over RAM" %
scenario._max_iters)
ERROR: line over 90 characters
#1284: FILE: tests/migration/guestperf/engine.py:255:
+ print("Switching to post-copy after %d iterations" %
scenario._post_copy_iters)
WARNING: line over 80 characters
#1293: FILE: tests/migration/guestperf/engine.py:263:
+ print("Pausing VM after %d iterations" %
scenario._pause_iters)
ERROR: line over 90 characters
#1651: 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
- Re: [Qemu-devel] [RFC 06/10] python: futurize -f lib2to3.fixes.fix_reduce, (continued)
- [Qemu-devel] [RFC 07/10] python: futurize -f lib2to3.fixes.fix_tuple_params, Eduardo Habkost, 2018/05/11
- [Qemu-devel] [RFC 08/10] python: futurize -f lib2to3.fixes.fix_renames, Eduardo Habkost, 2018/05/11
- [Qemu-devel] [RFC 09/10] python: futurize -f lib2to3.fixes.fix_except, Eduardo Habkost, 2018/05/11
- [Qemu-devel] [RFC 10/10] python: futurize -f lib2to3.fixes.fix_numliterals, Eduardo Habkost, 2018/05/11
- Re: [Qemu-devel] [RFC 00/10] [TESTING NEEDED] python: futurize --stage1 (Python 3 compatibility),
no-reply <=
- Re: [Qemu-devel] [RFC 00/10] [TESTING NEEDED] python: futurize --stage1 (Python 3 compatibility), Dr. David Alan Gilbert, 2018/05/14
- Re: [Qemu-devel] [RFC 00/10] [TESTING NEEDED] python: futurize --stage1 (Python 3 compatibility), Max Reitz, 2018/05/16
- Re: [Qemu-devel] [RFC 00/10] [TESTING NEEDED] python: futurize --stage1 (Python 3 compatibility), Fam Zheng, 2018/05/17