On 28/04/2019 18.10, no-re...@patchew.org wrote: > Patchew URL: https://patchew.org/QEMU/20190428155451.15653-1-th...@redhat.com/ > > > > Hi, > > This series seems to have some coding style problems. See output below for > more information: > > Type: series > Message-id: 20190428155451.15653-1-th...@redhat.com > Subject: [Qemu-devel] [PATCH v2 0/8] tests/qemu-iotests: Run basic iotests > during "make check" > > === 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 > From https://github.com/patchew-project/qemu > * [new tag] patchew/20190428155451.15653-1-th...@redhat.com -> > patchew/20190428155451.15653-1-th...@redhat.com > Switched to a new branch 'test' > 7bb09e08a3 tests: Run the iotests during "make check" again > 3d905a091e tests/qemu-iotests/group: Re-use the "auto" group for tests that > can always run > 565003ac8a tests/qemu-iotests: Remove the "_supported_os Linux" line from > many tests > fb27dba3d9 cirrus / travis: Add gnu-sed and bash for macOS and FreeBSD > 02ac8a4bf6 tests/qemu-iotests: Do not hard-code the path to bash > 34c6a891ea tests/qemu-iotests/check: Pick a default machine if necessary > d091edf7c1 tests/qemu-iotests/005: Add a sanity check for large sparse file > support > 3cf4caa193 tests/qemu-iotests: Fix output of qemu-io related tests > > === OUTPUT BEGIN === > 1/8 Checking commit 3cf4caa193cc (tests/qemu-iotests: Fix output of qemu-io > related tests) > 2/8 Checking commit d091edf7c1ef (tests/qemu-iotests/005: Add a sanity check > for large sparse file support) > 3/8 Checking commit 34c6a891ea6e (tests/qemu-iotests/check: Pick a default > machine if necessary) > 4/8 Checking commit 02ac8a4bf608 (tests/qemu-iotests: Do not hard-code the > path to bash) > 5/8 Checking commit fb27dba3d9f0 (cirrus / travis: Add gnu-sed and bash for > macOS and FreeBSD) > 6/8 Checking commit 565003ac8a70 (tests/qemu-iotests: Remove the > "_supported_os Linux" line from many tests) > 7/8 Checking commit 3d905a091e64 (tests/qemu-iotests/group: Re-use the "auto" > group for tests that can always run) > ERROR: trailing whitespace > #125: FILE: tests/qemu-iotests/group:115: > +093 $
Not an error: The trailing whitespace is currently required for the parsing code in the "check" script. Thomas
