Re: [Qemu-devel] [PATCH v1 00/42] current testing/next queue (podman, docker, ci)

[no-reply]

Patchew URL: https://patchew.org/QEMU/address@hidden/



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: address@hidden

=== 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/address@hidden/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to address@hidden