Message ID | 20190318162410.10183-1-mdroth@linux.vnet.ibm.com |
---|---|
State | New |
Headers | show |
Patchew URL: https://patchew.org/QEMU/20190318162410.10183-1-mdroth@linux.vnet.ibm.com/ Hi, This series seems to have some coding style problems. See output below for more information: Message-id: 20190318162410.10183-1-mdroth@linux.vnet.ibm.com Type: series Subject: [Qemu-devel] [PULL for-4.0 0/7] qemu-ga patch queue for 4.0 === TEST SCRIPT BEGIN === #!/bin/bash git rev-parse base > /dev/null || exit 0 git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram ./scripts/checkpatch.pl --mailback base.. === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 From https://github.com/patchew-project/qemu * [new tag] patchew/20190318162410.10183-1-mdroth@linux.vnet.ibm.com -> patchew/20190318162410.10183-1-mdroth@linux.vnet.ibm.com Switched to a new branch 'test' 6049403cd7 qmp: common 'id' handling & make QGA conform to QMP spec 7617b6d75f qga: process_event() simplification 780b66b189 qga: Fix guest-get-fsinfo PCI address collection in Windows 398c80ae8e qga-win: fix VSS build breakage due to unintended gnu99 C++ flag 232ff93fdd qga-win: include glib when building VSS DLL 5631cb464b qga-win: Adding support for Windows Server 2019 get-osinfo command 626bf490e5 qga: update docs with systemd suspend support info === OUTPUT BEGIN === 1/7 Checking commit 626bf490e544 (qga: update docs with systemd suspend support info) 2/7 Checking commit 5631cb464b54 (qga-win: Adding support for Windows Server 2019 get-osinfo command) 3/7 Checking commit 232ff93fddbb (qga-win: include glib when building VSS DLL) 4/7 Checking commit 398c80ae8e02 (qga-win: fix VSS build breakage due to unintended gnu99 C++ flag) 5/7 Checking commit 780b66b18953 (qga: Fix guest-get-fsinfo PCI address collection in Windows) ERROR: spaces required around that '&' (ctx:VxV) #312: FILE: qga/commands-win32.c:668: + &type, (PBYTE)&bus, size, NULL)) { ^ ERROR: spaces required around that '&' (ctx:VxV) #336: FILE: qga/commands-win32.c:680: + &type, (PBYTE)&addr, size, NULL)) { ^ ERROR: spaces required around that '&' (ctx:VxV) #348: FILE: qga/commands-win32.c:692: + &type, (PBYTE)&ui_slot, size, NULL)) { ^ total: 3 errors, 0 warnings, 419 lines checked Patch 5/7 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 6/7 Checking commit 7617b6d75fd5 (qga: process_event() simplification) 7/7 Checking commit 6049403cd793 (qmp: common 'id' handling & make QGA conform to QMP spec) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20190318162410.10183-1-mdroth@linux.vnet.ibm.com/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-devel@redhat.com
On Mon, 18 Mar 2019 at 16:24, Michael Roth <mdroth@linux.vnet.ibm.com> wrote: > > The following changes since commit 3c18a08cf387d44a351b9877497b5e0ea27c5140: > > Merge remote-tracking branch 'remotes/kraxel/tags/seabios-1.12.1-20190318-pull-request' into staging (2019-03-18 13:14:24 +0000) > > are available in the Git repository at: > > git://github.com/mdroth/qemu.git tags/qga-pull-2019-03-18-tag > > for you to fetch changes up to 4eaca8de268d74ac5daaf8938abcb69d37ba2889: > > qmp: common 'id' handling & make QGA conform to QMP spec (2019-03-18 10:48:06 -0500) > > ---------------------------------------------------------------- > qemu-ga patch queue for 4.0 > > * fix w32 build breakages with VSS enabled > * fix PCI topology reporting for Windows > * fix OS version reporting for Windows > * add systemd info to qga schema documentation > * add proper 'id' handling to QGA so it conforms to QMP spec > Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/4.0 for any user-visible changes. -- PMM