Message ID | 20170831082210.8362-1-stefanha@redhat.com |
---|---|
State | New |
Headers | show |
Hi, This series seems to have some coding style problems. See output below for more information: Message-id: 20170831082210.8362-1-stefanha@redhat.com Subject: [Qemu-devel] [PULL for-2.10 00/15] Block patches Type: series === 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 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/20170830215523.25278-1-f4bug@amsat.org -> patchew/20170830215523.25278-1-f4bug@amsat.org * [new tag] patchew/20170831082210.8362-1-stefanha@redhat.com -> patchew/20170831082210.8362-1-stefanha@redhat.com Switched to a new branch 'test' eaf80da3e4 qcow2: allocate cluster_cache/cluster_data on demand fab9c02727 qemu-doc: Add UUID support in initiator name 76e70ed0d9 tests: migration/guestperf Python 2.6 argparse compatibility 316bda7e7b docker.py: Python 2.6 argparse compatibility 0a7f8a625b scripts: add argparse module for Python 2.6 compatibility 3e6e26d514 misc: Remove unused Error variables f01d195374 oslib-posix: Print errors before aborting on qemu_alloc_stack() b2f08be812 throttle: Test the valid range of config values f1cd43f5e4 throttle: Make burst_length 64bit and add range checks 60981818d1 throttle: Make LeakyBucket.avg and LeakyBucket.max integer types 3032f687f4 throttle: Remove throttle_fix_bucket() / throttle_unfix_bucket() 73e3d4f547 throttle: Make throttle_is_valid() a bit less verbose 0ae5ba07ca throttle: Update the throttle_fix_bucket() documentation 9f52f0ea86 throttle: Fix wrong variable name in the header documentation 062842b9f2 nvme: Fix get/set number of queues feature, again === OUTPUT BEGIN === Checking PATCH 1/15: nvme: Fix get/set number of queues feature, again... Checking PATCH 2/15: throttle: Fix wrong variable name in the header documentation... Checking PATCH 3/15: throttle: Update the throttle_fix_bucket() documentation... Checking PATCH 4/15: throttle: Make throttle_is_valid() a bit less verbose... Checking PATCH 5/15: throttle: Remove throttle_fix_bucket() / throttle_unfix_bucket()... Checking PATCH 6/15: throttle: Make LeakyBucket.avg and LeakyBucket.max integer types... Checking PATCH 7/15: throttle: Make burst_length 64bit and add range checks... Checking PATCH 8/15: throttle: Test the valid range of config values... Checking PATCH 9/15: oslib-posix: Print errors before aborting on qemu_alloc_stack()... Checking PATCH 10/15: misc: Remove unused Error variables... Checking PATCH 11/15: scripts: add argparse module for Python 2.6 compatibility... ERROR: trailing whitespace #118: FILE: COPYING.PYTHON:93: +Reserved" are retained in Python alone or in any derivative version $ total: 1 errors, 0 warnings, 2676 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 12/15: docker.py: Python 2.6 argparse compatibility... Checking PATCH 13/15: tests: migration/guestperf Python 2.6 argparse compatibility... Checking PATCH 14/15: qemu-doc: Add UUID support in initiator name... Checking PATCH 15/15: qcow2: allocate cluster_cache/cluster_data on demand... === OUTPUT END === Test command exited with code: 1 --- Email generated automatically by Patchew [http://patchew.org/]. Please send your feedback to patchew-devel@freelists.org
On 08/31/2017 03:21 AM, Stefan Hajnoczi wrote:
> The following changes since commit 248b23735645f7cbb503d9be6f5bf825f2a603ab:
Subject line should probably say 2.11, not 2.10 :)
On 31 August 2017 at 09:21, Stefan Hajnoczi <stefanha@redhat.com> wrote: > The following changes since commit 248b23735645f7cbb503d9be6f5bf825f2a603ab: > > Update version for v2.10.0-rc4 release (2017-08-24 17:34:26 +0100) > > are available in the git repository at: > > git://github.com/stefanha/qemu.git tags/block-pull-request > > for you to fetch changes up to 3e4c705212abfe8c9882a00beb2d1466a8a53cec: > > qcow2: allocate cluster_cache/cluster_data on demand (2017-08-30 18:02:10 +0100) > > ---------------------------------------------------------------- > > ---------------------------------------------------------------- Applied, thanks. -- PMM