Patchew URL: https://patchew.org/QEMU/20190904203013.9028-1-alex.ben...@linaro.org/
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Subject: [Qemu-devel] [PATCH v1 00/42] current testing/next queue (podman, docker, ci) Message-id: 20190904203013.9028-1-alex.ben...@linaro.org === 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 Switched to a new branch 'test' 0fbbd2b tests/docker: don't always encoding for subprocess.check_output 5b2ad81 tests/docker: --disable-libssh on ubuntu1804 builds c3ee1ae Fedora images: use URLs from stable "archives.fedoraproject.org" 942403a .travis.yml: Cache Linux/Clang jobs together 5e27a63 .travis.yml: Cache Linux/GCC 'non-debug profile' jobs together eb3387c .travis.yml: Cache Linux/GCC 'debug profile' jobs together b88f2bb .travis.yml: Document how the build matrix use caches a655c67 .travis.yml: Enable ccache on OSX 5ffee09 .travis.yml: Improve ccache use aff75d2 .travis.yml: Cache Avocado cache f16c75b .travis.yml: Cache Python PIP packages 0833369 .travis.yml: Increase cache timeout from 3min to 20min 0b7ee1a .travis.yml: Enable multiple caching features cc73e57 configure: check if --no-pie is supported first 08b9a33 tests/docker: add more images to PARTIAL_IMAGES when not on x86_64 5cc1f1f tests/docker: use --arch-only for installing deps 0134450 tests/docker: add debian-amd64-cross for non-x86 hosts 2a70b8d tests/docker: avoid $SHELL invoke bash directly 3d28ab1 tests/docker: add debian9-mxe to DEBIAN_PARTIAL_IMAGES 324696f tests/docker: add debian-xtensa-cross to DEBIAN_PARTIAL_IMAGES c465faa tests/docker: drop powerpc-user image for QEMU cross builds 213eeb6 tests/docker: drop debian-sid image 9f2f21c tests/docker: move our ppc64 cross compile to Buster f898a09 tests/docker: move our riscv64 cross compile to Buster a60a7b9 tests/docker: move our mips64 cross compile to Buster 2e05203 tests/docker: move our sh4 cross compile to Buster dbcdf48 tests/docker: move our sparc64 cross compile to Buster 1c3befa tests/docker: move our m68k cross compile to Buster 654bcae tests/docker: move our HPPA cross compile to Buster 0e725a4 tests/docker: move our Alpha cross compile to Buster ca3a13e tests/docker: move our powerpc cross compile to Buster b92b820 tests/docker: move our arm64 cross compile to Buster 388f703 tests/docker: add Buster to DOCKER_PARTIAL_IMAGES 102d04a tests/docker: set DEF_TARGET_LIST for some containers fc29026 tests/docker: move DEF_TARGET_LIST setting to common.rc 56cd636 tests/tcg: add .gitignore for in source builds 3bad540 tests/tcg: move configuration to a sub-shell script 5f89352 tests/tcg: cleanup Makefile inclusions e9af276 tests/tcg: use EXTRA_CFLAGS everywhere dac6b94 tests/docker: fix "cc" command to work with podman eefeff4 configure: clean-up container cross compile detect 1cd7a8b hw/misc: Mark most objects as "common" code to speed up compilation a litte bit === OUTPUT BEGIN === 1/42 Checking commit 1cd7a8bf605d (hw/misc: Mark most objects as "common" code to speed up compilation a litte bit) 2/42 Checking commit eefeff4f57f7 (configure: clean-up container cross compile detect) 3/42 Checking commit dac6b9434eb1 (tests/docker: fix "cc" command to work with podman) 4/42 Checking commit e9af276e922a (tests/tcg: use EXTRA_CFLAGS everywhere) 5/42 Checking commit 5f89352ea324 (tests/tcg: cleanup Makefile inclusions) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #84: rename from tests/tcg/Makefile.probe total: 0 errors, 1 warnings, 341 lines checked Patch 5/42 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 6/42 Checking commit 3bad54027509 (tests/tcg: move configuration to a sub-shell script) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #706: deleted file mode 100644 WARNING: line over 80 characters #920: FILE: tests/tcg/configure.sh:174: + container_cross_cc=/opt/2018.02/xtensa-dc232b-elf/bin/xtensa-dc232b-elf-gcc WARNING: line over 80 characters #926: FILE: tests/tcg/configure.sh:180: + echo "# Automatically generated by configure - do not modify" > $config_target_mak ERROR: line over 90 characters #953: FILE: tests/tcg/configure.sh:207: + if ! do_compiler "$target_compiler" $target_compiler_cflags -o $TMPE $TMPC -static ; then WARNING: line over 80 characters #955: FILE: tests/tcg/configure.sh:209: + if ! do_compiler "$target_compiler" $target_compiler_cflags -o $TMPE $TMPC ; then WARNING: line over 80 characters #968: FILE: tests/tcg/configure.sh:222: + if test $got_cross_cc = no && test "$docker" != no && test -n "$container_image"; then total: 1 errors, 5 warnings, 863 lines checked Patch 6/42 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 7/42 Checking commit 56cd636ccbbb (tests/tcg: add .gitignore for in source builds) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #14: new file mode 100644 total: 0 errors, 1 warnings, 5 lines checked Patch 7/42 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 8/42 Checking commit fc29026cb47e (tests/docker: move DEF_TARGET_LIST setting to common.rc) 9/42 Checking commit 102d04a5c59a (tests/docker: set DEF_TARGET_LIST for some containers) 10/42 Checking commit 388f70343172 (tests/docker: add Buster to DOCKER_PARTIAL_IMAGES) 11/42 Checking commit b92b82082115 (tests/docker: move our arm64 cross compile to Buster) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #57: deleted file mode 100644 total: 0 errors, 1 warnings, 40 lines checked Patch 11/42 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 12/42 Checking commit ca3a13e95f3c (tests/docker: move our powerpc cross compile to Buster) 13/42 Checking commit 0e725a419d31 (tests/docker: move our Alpha cross compile to Buster) 14/42 Checking commit 654bcae3bead (tests/docker: move our HPPA cross compile to Buster) 15/42 Checking commit 1c3befa14744 (tests/docker: move our m68k cross compile to Buster) 16/42 Checking commit dbcdf486beab (tests/docker: move our sparc64 cross compile to Buster) 17/42 Checking commit 2e0520300fdf (tests/docker: move our sh4 cross compile to Buster) 18/42 Checking commit a60a7b9d64be (tests/docker: move our mips64 cross compile to Buster) 19/42 Checking commit f898a09c7fc6 (tests/docker: move our riscv64 cross compile to Buster) 20/42 Checking commit 9f2f21c480e7 (tests/docker: move our ppc64 cross compile to Buster) 21/42 Checking commit 213eeb6def1e (tests/docker: drop debian-sid image) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #28: deleted file mode 100644 total: 0 errors, 1 warnings, 8 lines checked Patch 21/42 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 22/42 Checking commit c465faafdef8 (tests/docker: drop powerpc-user image for QEMU cross builds) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #35: deleted file mode 100644 total: 0 errors, 1 warnings, 14 lines checked Patch 22/42 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 23/42 Checking commit 324696f9c362 (tests/docker: add debian-xtensa-cross to DEBIAN_PARTIAL_IMAGES) 24/42 Checking commit 3d28ab1cff6e (tests/docker: add debian9-mxe to DEBIAN_PARTIAL_IMAGES) 25/42 Checking commit 2a70b8df701a (tests/docker: avoid $SHELL invoke bash directly) 26/42 Checking commit 01344502cbaa (tests/docker: add debian-amd64-cross for non-x86 hosts) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #32: new file mode 100644 total: 0 errors, 1 warnings, 35 lines checked Patch 26/42 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 27/42 Checking commit 5cc1f1fd5605 (tests/docker: use --arch-only for installing deps) 28/42 Checking commit 08b9a3310119 (tests/docker: add more images to PARTIAL_IMAGES when not on x86_64) 29/42 Checking commit cc73e576887a (configure: check if --no-pie is supported first) 30/42 Checking commit 0b7ee1a3bfea (.travis.yml: Enable multiple caching features) 31/42 Checking commit 083336916ba0 (.travis.yml: Increase cache timeout from 3min to 20min) 32/42 Checking commit f16c75bc40ac (.travis.yml: Cache Python PIP packages) 33/42 Checking commit aff75d22d044 (.travis.yml: Cache Avocado cache) 34/42 Checking commit 5ffee09da795 (.travis.yml: Improve ccache use) 35/42 Checking commit a655c67011d0 (.travis.yml: Enable ccache on OSX) 36/42 Checking commit b88f2bb3bd9b (.travis.yml: Document how the build matrix use caches) 37/42 Checking commit eb3387cf994d (.travis.yml: Cache Linux/GCC 'debug profile' jobs together) 38/42 Checking commit 5e27a63aee94 (.travis.yml: Cache Linux/GCC 'non-debug profile' jobs together) 39/42 Checking commit 942403a1ad9c (.travis.yml: Cache Linux/Clang jobs together) 40/42 Checking commit c3ee1aec4faf (Fedora images: use URLs from stable "archives.fedoraproject.org") 41/42 Checking commit 5b2ad81bb4a1 (tests/docker: --disable-libssh on ubuntu1804 builds) 42/42 Checking commit 0fbbd2b57dd7 (tests/docker: don't always encoding for subprocess.check_output) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20190904203013.9028-1-alex.ben...@linaro.org/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-de...@redhat.com
