Re: [Qemu-devel] [PATCH v2 00/31] Updated Travis Queue

[no-reply]

Hi,

This series seems to have some coding style problems. See output below for
more information:

Subject: [Qemu-devel] [PATCH v2 00/31] Updated Travis Queue
Message-id: address@hidden
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
 - [tag update]      patchew/address@hidden -> patchew/address@hidden
 * [new tag]         patchew/address@hidden -> patchew/address@hidden
Switched to a new branch 'test'
ccf8b79 docker: add debian Ports base image
1d50216 shippable: add win32/64 targets
2d60fa6 docker: add MXE (M cross environment) base image for MinGW-w64
9ac23ef shippable: add mips64el targets
6359551 docker: add debian/mips64el image
334ba00 shippable: use debian/mips[eb] targets
01c95b4 docker: add debian/mips[eb] images
2e63e0d shippable: add powerpc target
d2efe10 docker: add debian/powerpc based on Jessie
967070c docker: add 'apt-fake' script which generate fake debian packages
aaa08ff docker: add qemu:debian-jessie based on outdated jessie release
c21ee29 util/cacheinfo: add missing include
65ce350 shippable: add x86_64 targets
2b36cc5 shippable: add ppc64el targets
0a38183 shippable: add armel targets
246c517 docker: enable nettle to extend code coverage on arm64
0bc3dcb docker: enable gcrypt to extend code coverage on amd64
df95408 docker: enable netmap to extend code coverage on amd64
951854e docker: enable virgl to extend code coverage on amd64
614d95e docker: add debian/amd64 based on Stretch
a6f5e01 docker: add debian/ppc64el based on Stretch
88e3839 docker: add debian/armel based on Stretch
96ac7ec docker: optimize debian9 base image
a8da518 docker: remove packages now dependent of qemu in Stretch
e0219b2 docker: debian/s390x no more in unstable, now available in Stretch
3f63830 docker: add common packages to debian base
77d452a docker: rename debian stable -> 9 (Stretch)
fe1bebf docker: add debug tools to travis.docker
0738a47 docker: include python-yaml in travis.docker
41e0862 travis: move make -j flag out of script
6ca5c9a docker: allow customizing Travis global_env variables

=== OUTPUT BEGIN ===
Checking PATCH 1/31: docker: allow customizing Travis global_env variables...
Checking PATCH 2/31: travis: move make -j flag out of script...
Checking PATCH 3/31: docker: include python-yaml in travis.docker...
Checking PATCH 4/31: docker: add debug tools to travis.docker...
Checking PATCH 5/31: docker: rename debian stable -> 9 (Stretch)...
Checking PATCH 6/31: docker: add common packages to debian base...
Checking PATCH 7/31: docker: debian/s390x no more in unstable, now available in 
Stretch...
Checking PATCH 8/31: docker: remove packages now dependent of qemu in Stretch...
Checking PATCH 9/31: docker: optimize debian9 base image...
Checking PATCH 10/31: docker: add debian/armel based on Stretch...
Checking PATCH 11/31: docker: add debian/ppc64el based on Stretch...
Checking PATCH 12/31: docker: add debian/amd64 based on Stretch...
Checking PATCH 13/31: docker: enable virgl to extend code coverage on amd64...
Checking PATCH 14/31: docker: enable netmap to extend code coverage on amd64...
Checking PATCH 15/31: docker: enable gcrypt to extend code coverage on amd64...
Checking PATCH 16/31: docker: enable nettle to extend code coverage on arm64...
Checking PATCH 17/31: shippable: add armel targets...
Checking PATCH 18/31: shippable: add ppc64el targets...
Checking PATCH 19/31: shippable: add x86_64 targets...
Checking PATCH 20/31: util/cacheinfo: add missing include...
Checking PATCH 21/31: docker: add qemu:debian-jessie based on outdated jessie 
release...
Checking PATCH 22/31: docker: add 'apt-fake' script which generate fake debian 
packages...
Checking PATCH 23/31: docker: add debian/powerpc based on Jessie...
Checking PATCH 24/31: shippable: add powerpc target...
Checking PATCH 25/31: docker: add debian/mips[eb] images...
Checking PATCH 26/31: shippable: use debian/mips[eb] targets...
Checking PATCH 27/31: docker: add debian/mips64el image...
Checking PATCH 28/31: shippable: add mips64el targets...
Checking PATCH 29/31: docker: add MXE (M cross environment) base image for 
MinGW-w64...
ERROR: trailing whitespace
#135: FILE: tests/docker/dockerfiles/debian8-mxe.docker:18:
+ENV PATH $PATH:/usr/lib/mxe/usr/bin/ $

total: 1 errors, 0 warnings, 97 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 30/31: shippable: add win32/64 targets...
Checking PATCH 31/31: docker: add debian Ports base image...
=== OUTPUT END ===

Test command exited with code: 1


---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to address@hidden