[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 00/60] json: Fixes, error reporting improveme
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2 00/60] json: Fixes, error reporting improvements, cleanups |
Date: |
Mon, 20 Aug 2018 13:59:53 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Fam Zheng <address@hidden> writes:
> On Mon, 08/20 10:31, Markus Armbruster wrote:
>> address@hidden writes:
>>
>> > Hi,
>> >
>> > This series seems to have some coding style problems. See output below for
>> > more information:
>> [...]
>> > Checking PATCH 4/60: qmp-cmd-test: Split off qmp-test...
>> > WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
>> > #48:
>> > new file mode 100644
>>
>> False positive: the patch does update MAINTAINERS. Surprising, since
>> commit 1a6fad0c3bd "checkpatch: reduce MAINTAINERS update message
>> frequency" suppresses this. Fam, is patchew running an outdated version
>> of checkpatch?
>
> I'm sure Patchew is using the latest checkpatch here. This is basically /me
> reproducing what it does:
>
> address@hidden:~/work/qemu [master]$ git fetch
> https://github.com/patchew-project/qemu tags/patchew/address@hidden
>>From https://github.com/patchew-project/qemu
> * tag patchew/address@hidden -> FETCH_HEAD
> address@hidden:~/work/qemu [master]$ git checkout FETCH_HEAD
> Note: checking out 'FETCH_HEAD'.
>
> You are in 'detached HEAD' state. You can look around, make experimental
> changes and commit them, and you can discard any commits you make in this
> state without impacting any branches by performing another checkout.
>
> If you want to create a new branch to retain commits you create, you may
> do so (now or later) by using -b with the checkout command again. Example:
>
> git checkout -b <new-branch-name>
>
> HEAD is now at 884c1e75d7 json: Support %% in JSON strings when interpolating
> address@hidden:~/work/qemu$ git log --oneline | grep 'qmp-cmd-test: Split off
> qmp-test'
> b9ac2b614b qmp-cmd-test: Split off qmp-test
> address@hidden:~/work/qemu$ git show --format=email b9ac2b614b |
> ./scripts/checkpatch.pl -
> WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
> #48:
> new file mode 100644
>
> total: 0 errors, 1 warnings, 460 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.
> address@hidden:~/work/qemu$ git log --oneline ./scripts/checkpatch.pl | grep
> 1a6fad0
> 1a6fad0c3b checkpatch: reduce MAINTAINERS update message frequency
Okay, I can reproduce the warning the way patchew runs checkpatch:
$ git-show --format=email 4552d915d3 | ./scripts/checkpatch.pl -
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#47:
new file mode 100644
total: 0 errors, 1 warnings, 460 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.
However:
$ git-format-patch -1 4552d915d3
0001-qmp-cmd-test-Split-off-qmp-test.patch
$ ./scripts/checkpatch.pl 0001-qmp-cmd-test-Split-off-qmp-test.patch
total: 0 errors, 0 warnings, 460 lines checked
0001-qmp-cmd-test-Split-off-qmp-test.patch has no obvious style problems
and is ready for submission.
The difference between the two: git-show --format=email lacks diffstat.
$ git-show --format=email 4552d915d3 | diff -
0001-qmp-cmd-test-Split-off-qmp-test.patch
11a12,18
> ---
> MAINTAINERS | 1 +
> tests/Makefile.include | 3 +
> tests/qmp-cmd-test.c | 213 +++++++++++++++++++++++++++++++++++++++++
> tests/qmp-test.c | 193 +------------------------------------
> 4 files changed, 219 insertions(+), 191 deletions(-)
> create mode 100644 tests/qmp-cmd-test.c
497a505,507
> --
> 2.17.1
>
- Re: [Qemu-devel] [PATCH v2 57/60] tests/drive_del-test: Fix harmless JSON interpolation bug, (continued)
- [Qemu-devel] [PATCH v2 48/60] json: Enforce token count and size limits more tightly, Markus Armbruster, 2018/08/17
- [Qemu-devel] [PATCH v2 56/60] docs/interop/qmp-spec: How to force known good parser state, Markus Armbruster, 2018/08/17
- [Qemu-devel] [PATCH v2 54/60] qobject: Drop superfluous includes of qemu-common.h, Markus Armbruster, 2018/08/17
- [Qemu-devel] [PATCH v2 47/60] qjson: Have qobject_from_json() & friends reject empty and blank, Markus Armbruster, 2018/08/17
- [Qemu-devel] [PATCH v2 55/60] json: Clean up headers, Markus Armbruster, 2018/08/17
- Re: [Qemu-devel] [PATCH v2 00/60] json: Fixes, error reporting improvements, cleanups, no-reply, 2018/08/18