[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCHv4] tests/tcg: fix build
From: |
Catalin Patulea |
Subject: |
Re: [Qemu-devel] [PATCHv4] tests/tcg: fix build |
Date: |
Tue, 16 Oct 2012 15:57:40 -0400 |
On Tue, Oct 16, 2012 at 3:54 PM, Peter Maydell <address@hidden> wrote:
> This patch isn't in the right format to apply, I'm afraid.
> Major problem:
> * missing signed-off-by line: we can't accept any patch
> without one
> Minor problem:
> * the actual commit message is below the first '---' line,
> so 'git am' won't see it. Commit messages go above '---',
> "what I changed in this version' notes below.
I saw this format (notes, commit message, diffstat) on another list
and assumed git am could handle this properly.
It just seems strange to write the note somewhere *below* the commit
message - the notes should be the first thing people read, and below
them is the complete proposed patch. Is there any way to achieve that?
Meanwhile, I'll reformat and resend the patch.
- [Qemu-devel] [PATCHv3] tests/tcg: fix build, Catalin Patulea, 2012/10/16
- Re: [Qemu-devel] [PATCHv3] tests/tcg: fix build, Peter Maydell, 2012/10/16
- [Qemu-devel] [PATCHv4] tests/tcg: fix build, Catalin Patulea, 2012/10/16
- [Qemu-devel] [PATCHv5] tests/tcg: fix build, Catalin Patulea, 2012/10/16
- Re: [Qemu-devel] [PATCHv5] tests/tcg: fix build, Peter Maydell, 2012/10/16
- Re: [Qemu-devel] [PATCHv5] tests/tcg: fix build, Aurelien Jarno, 2012/10/19