Hi, This series seems to have some coding style problems. See output below for more information:
Subject: [Qemu-devel] [PULL v2 00/37] Staging patches Message-id: [email protected] 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/[email protected] -> patchew/[email protected] t [tag update] patchew/[email protected] -> patchew/[email protected] * [new tag] patchew/[email protected] -> patchew/[email protected] Switched to a new branch 'test' 274b0b33cb buildsys: Move rdma libs to per object dee13ca0ca buildsys: Move brlapi libs to per object 51486e4caf buildsys: Move usb redir cflags/libs to per object e2d2141400 buildsys: Move libusb cflags/libs to per object e653260901 buildsys: Move libcacard cflags/libs to per object de80133478 buildsys: Move curese cflags/libs to per object 4da943d9c6 buildsys: Move audio libs to per object c05be913ea buildsys: Move vnc cflags/libs to per object 066db85c1a buildsys: Move sdl cflags/libs to per object 50200dfb4c buildsys: Move gtk/vte cflags/libs to per object 0832346d7d buildsys: Move vde libs to per object 5cad9f269f vl: Don't include vde header d642c77e93 docker: Drop 'set -e' from run script 05cb7137d9 docker: Add test-block f829f5d141 docker: Add nettle-devel to fedora image ca0c6ffecc docker: Use unconfined security profile c5a6c192c7 docker: Add test_fail and prep_fail ee94d2b7c9 docker: Fix return code of build_qemu() 0717923a55 docker: Use archive-source.py b6a02aee1a tests: Add README for vm tests f0bfdb4c3b MAINTAINERS: Add tests/vm entry 133ce1feb5 Makefile: Add rules to run vm tests f88480f6c8 tests: Add OpenBSD image d05508ab23 tests: Add NetBSD image 8370cb8f2f tests: Add FreeBSD image a8b70bfa3b tests: Add ubuntu.i386 image 413861ba2d tests: Add vm test lib e10c67002d scripts: Add archive-source.sh c353b7acd2 qemu.py: Add "wait()" method fac58fe6c0 gitignore: Ignore vm test images 1022469bf7 tests/docker: Clean up paths e6dde9693e docker: Enable features explicitly in test-full f08e8a2903 docker: Update ubuntu image 6c2c01bbe2 docker: reduce noise when building travis.docker f63179c5a0 docker: don't install device-tree-compiler build-deps in travis.docker f9d3a9d011 docker: docker.py make --no-cache skip checksum test 3695d96c65 docker: ensure NOUSER for travis images === OUTPUT BEGIN === Checking PATCH 1/37: docker: ensure NOUSER for travis images... Checking PATCH 2/37: docker: docker.py make --no-cache skip checksum test... Checking PATCH 3/37: docker: don't install device-tree-compiler build-deps in travis.docker... Checking PATCH 4/37: docker: reduce noise when building travis.docker... Checking PATCH 5/37: docker: Update ubuntu image... Checking PATCH 6/37: docker: Enable features explicitly in test-full... Checking PATCH 7/37: tests/docker: Clean up paths... Checking PATCH 8/37: gitignore: Ignore vm test images... Checking PATCH 9/37: qemu.py: Add "wait()" method... Checking PATCH 10/37: scripts: Add archive-source.sh... Checking PATCH 11/37: tests: Add vm test lib... ERROR: line over 90 characters #86: FILE: tests/vm/basevm.py:60: +ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABAQCikC46WYtXotUd0UGPz9547Aj0KqC4gk+nt4BBJm86IHgCD9FygSGX9EFutXlhz9KZIPg9Okk7+IzXRHCWI2MNvhrcjyrezKREm71z08j9iwfxY3340fY2Mo+0khwpO7bzsgzkljHIHqcOg7MgttPInVMNH/EfqpgR8EDKJuWCB2Ny+EBFN/3dAiff0X/EvKle9PUrY70EkSycnyURS8HZReEqj8lN9J5kXzA8F6jBo/0Q42Ttv6e4k5YcaDrwmLrBWLra2PCXZLNyHqXEiFkGmdXtA1Eox9gc/p4jIXim6xrPNmpN6WyrrEjaCF5xYvNv8wXkD6uSWwbHYU24lIAn qemu-vm-key WARNING: line over 80 characters #100: FILE: tests/vm/basevm.py:74: + self._tmpdir = tempfile.mkdtemp(prefix="vm-test-", suffix=".tmp", dir=".") WARNING: line over 80 characters #191: FILE: tests/vm/basevm.py:165: + logging.debug("Creating archive %s for src_dir dir: %s", tarfile, src_dir) WARNING: line over 80 characters #196: FILE: tests/vm/basevm.py:170: + "file=%s,if=none,id=%s,cache=writeback,format=raw" % \ WARNING: line over 80 characters #199: FILE: tests/vm/basevm.py:173: + "virtio-blk,drive=%s,serial=%s,bootindex=1" % (name, name)] ERROR: line over 90 characters #246: FILE: tests/vm/basevm.py:220: + VM test utility. Exit codes: 0 = success, 1 = command line error, 2 = environment initialization failed, 3 = test command failed""") WARNING: line over 80 characters #253: FILE: tests/vm/basevm.py:227: + parser.add_option("--jobs", type=int, default=multiprocessing.cpu_count() / 2, total: 2 errors, 5 warnings, 276 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/37: tests: Add ubuntu.i386 image... Checking PATCH 13/37: tests: Add FreeBSD image... Checking PATCH 14/37: tests: Add NetBSD image... Checking PATCH 15/37: tests: Add OpenBSD image... Checking PATCH 16/37: Makefile: Add rules to run vm tests... Checking PATCH 17/37: MAINTAINERS: Add tests/vm entry... Checking PATCH 18/37: tests: Add README for vm tests... Checking PATCH 19/37: docker: Use archive-source.py... Checking PATCH 20/37: docker: Fix return code of build_qemu()... Checking PATCH 21/37: docker: Add test_fail and prep_fail... Checking PATCH 22/37: docker: Use unconfined security profile... Checking PATCH 23/37: docker: Add nettle-devel to fedora image... Checking PATCH 24/37: docker: Add test-block... Checking PATCH 25/37: docker: Drop 'set -e' from run script... Checking PATCH 26/37: vl: Don't include vde header... Checking PATCH 27/37: buildsys: Move vde libs to per object... Checking PATCH 28/37: buildsys: Move gtk/vte cflags/libs to per object... Checking PATCH 29/37: buildsys: Move sdl cflags/libs to per object... Checking PATCH 30/37: buildsys: Move vnc cflags/libs to per object... Checking PATCH 31/37: buildsys: Move audio libs to per object... Checking PATCH 32/37: buildsys: Move curese cflags/libs to per object... Checking PATCH 33/37: buildsys: Move libcacard cflags/libs to per object... Checking PATCH 34/37: buildsys: Move libusb cflags/libs to per object... Checking PATCH 35/37: buildsys: Move usb redir cflags/libs to per object... Checking PATCH 36/37: buildsys: Move brlapi libs to per object... Checking PATCH 37/37: buildsys: Move rdma libs to per object... === OUTPUT END === Test command exited with code: 1 --- Email generated automatically by Patchew [http://patchew.org/]. Please send your feedback to [email protected]
