Message ID | 20170815132746.12540-1-stefanha@redhat.com |
---|---|
State | New |
Headers | show |
Hi, This series seems to have some coding style problems. See output below for more information: Message-id: 20170815132746.12540-1-stefanha@redhat.com Subject: [Qemu-devel] [PULL 0/2] Tracing patches 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/20170814231552.24593-1-pbonzini@redhat.com -> patchew/20170814231552.24593-1-pbonzini@redhat.com t [tag update] patchew/20170815130502.8736-1-stefanha@redhat.com -> patchew/20170815130502.8736-1-stefanha@redhat.com Switched to a new branch 'test' c8a963888c simpletrace: fix flight recorder --no-header option 272f609607 trace: use static event ID mapping in simpletrace.stp === OUTPUT BEGIN === Checking PATCH 1/2: trace: use static event ID mapping in simpletrace.stp... Checking PATCH 2/2: simpletrace: fix flight recorder --no-header option... ERROR: line over 90 characters #42: FILE: scripts/simpletrace.py:101: + """Deserialize trace records from a file, yielding record tuples (event_num, timestamp, pid, arg1, ..., arg6). total: 1 errors, 0 warnings, 46 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. === OUTPUT END === Test command exited with code: 1 --- Email generated automatically by Patchew [http://patchew.org/]. Please send your feedback to patchew-devel@freelists.org
On 15 August 2017 at 14:27, Stefan Hajnoczi <stefanha@redhat.com> wrote: > The following changes since commit 5681da292242550f37ba4c03f46a8a6f8ee9278a: > > Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20170815' into staging (2017-08-15 09:39:14 +0100) > > are available in the git repository at: > > git://github.com/stefanha/qemu.git tags/tracing-pull-request > > for you to fetch changes up to 840d8351774664d01c328d31ed33b0e2d85c866e: > > simpletrace: fix flight recorder --no-header option (2017-08-15 12:50:29 +0100) > > ---------------------------------------------------------------- > > ---------------------------------------------------------------- > > Stefan Hajnoczi (2): > trace: use static event ID mapping in simpletrace.stp > simpletrace: fix flight recorder --no-header option > > scripts/simpletrace.py | 24 +++++++++++++++------ > scripts/tracetool/format/simpletrace_stap.py | 31 ++-------------------------- > 2 files changed, 20 insertions(+), 35 deletions(-) Applied, thanks. -- PMM