git am and the wrong chunk of ---

3 messages, 3 authors, 2016-06-15 · open the first message on its own page

git am and the wrong chunk of ---

From: "H. Peter Anvin" <hpa@zytor.com>
Date: 2016-06-15 22:54:27

Hello,

I have some contributors who consistently put their commentary *before* 
the "---" line rather than *after* it, presumably with the notion that 
it is some kind of "cover text".  This messes with "git am", and so I 
end up having to edit those posts manually.

I have tried git am --scissors and it doesn't seem to solve the problem.

Is there any other option which can be used to automatically process 
such a patch?

	-hpa

-- 
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel.  I don't speak on their behalf.

Re: git am and the wrong chunk of ---

From: Jeff King <hidden>
Date: 2016-06-15 22:54:27

On Thu, Aug 09, 2012 at 05:13:51PM -0700, H. Peter Anvin wrote:
I have some contributors who consistently put their commentary
*before* the "---" line rather than *after* it, presumably with the
notion that it is some kind of "cover text".  This messes with "git
am", and so I end up having to edit those posts manually.

I have tried git am --scissors and it doesn't seem to solve the problem.

Is there any other option which can be used to automatically process
such a patch?
If I understand your issue, somebody is writing:


    From: them
    To: you
    Date: ...
    Subject: [PATCH] subject line

    commit message body
    ....

    some cover letter material that should go below the "---"
    ---
      [diffstat + diff]


How do you know when the commit message body ends, and the cover letter
begins? We already have two machine-readable formats for separating the
two ("---" after the commit message, and "-- >8 --" scissors before). Is
there some machine-readable hint? Is it always the paragraph before the
"---"? Chopping that off unconditionally seems like a dangerous
heuristic.

-Peff

Re: git am and the wrong chunk of ---

From: Andreas Ericsson <hidden>
Date: 2016-06-15 22:54:27

On 08/10/2012 12:36 PM, Jeff King wrote:
On Thu, Aug 09, 2012 at 05:13:51PM -0700, H. Peter Anvin wrote:
quoted
I have some contributors who consistently put their commentary
*before* the "---" line rather than *after* it, presumably with the
notion that it is some kind of "cover text".  This messes with "git
am", and so I end up having to edit those posts manually.

I have tried git am --scissors and it doesn't seem to solve the problem.

Is there any other option which can be used to automatically process
such a patch?
If I understand your issue, somebody is writing:


     From: them
     To: you
     Date: ...
     Subject: [PATCH] subject line

     commit message body
     ....

     some cover letter material that should go below the "---"
     ---
       [diffstat + diff]


How do you know when the commit message body ends, and the cover letter
begins? We already have two machine-readable formats for separating the
two ("---" after the commit message, and "-- >8 --" scissors before). Is
there some machine-readable hint? Is it always the paragraph before the
"---"? Chopping that off unconditionally seems like a dangerous
heuristic.
End of SOB lines might be a good cutoff, if they're present. I've never
seen anyone put commit message text below them anyway.

-- 
Andreas Ericsson                   andreas.ericsson@op5.se
OP5 AB                             www.op5.se
Tel: +46 8-230225                  Fax: +46 8-230231

Considering the successes of the wars on alcohol, poverty, drugs and
terror, I think we should give some serious thought to declaring war
on peace.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help