[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] docker: add sanitizers back to clang build
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH] docker: add sanitizers back to clang build |
Date: |
Wed, 18 Sep 2019 10:38:26 +0800 |
User-agent: |
Mutt/1.12.1 (2019-06-15) |
On Thu, 09/12 19:07, John Snow wrote:
>
>
> On 9/11/19 9:52 PM, address@hidden wrote:
> > Patchew URL: https://patchew.org/QEMU/address@hidden/
> >
> >
> >
> > Hi,
> >
> > This series seems to have some coding style problems. See output below for
> > more information:
> >
> > Subject: [Qemu-devel] [PATCH] docker: add sanitizers back to clang build
> > Message-id: address@hidden
> > Type: series
> >
> > === 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 ===
> >
> > From https://github.com/patchew-project/qemu
> > * [new tag] patchew/address@hidden -> patchew/address@hidden
> > Switched to a new branch 'test'
> > 96d44b9 docker: add sanitizers back to clang build
> >
> > === OUTPUT BEGIN ===
> > ERROR: Missing Signed-off-by: line(s)
>
> GDI.
>
> I keep adding this to my configuration files, but it keeps "falling
> off", somehow.
>
> I have some patches in the works for stgit where I'm going to work
> through some test cases for setting profile variables and try to fix this.
>
> In the meantime:
>
> Signed-off-by: John Snow <address@hidden>
Isn't this because you inserted a '---' line in the middle of the commit
message so the part after it is ditched by 'git am'?
This feels a bit hard to catch, wondering what is in the works. :)
Fam