git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Git Bug - diff in commit message.

From
Thomas Rast <trast@student.ethz.ch>
Date
Sep 9, 2011, 00:56 UTC
Message-ID
<201109090256.36306.trast@student.ethz.ch>
In-Reply-To
<CAMOZ1BtbpbG+19G6Hfau_2_F5L3Ad+x-Payd9aKajJxU_V_tyA@mail.gmail.com>
Michael Witten wrote:
Show 14 quoted lines
> On Thu, Sep 8, 2011 at 14:49, anikey <arty.anikey@gmail.com> wrote:
> > Hi, peops. I'm pretty much sure that's a bug.
> >
> > What I did was putting git diff (i needed to tell people that for my changes
> > to start working they needed to aplly message-inline patch to some code
> > which was not under git) in commit message. Like adding:
> 
> It would appear that `git rebase' is in fact producing patches with
> `git format-patch' and then applying the resulting patches with `git
> am', which gets confused by your inline diff; this can be clearly seen
> in the `git-rebase--am[.sh]' file.
> 
> Perhaps `git rebase' should be reimplemented to use `git cherry-pick',
> or does that suffer from the same problem?

It doesn't, since it uses a three-way merge. But there's no real reason to reimplement anything; just use an interactive rebase, it uses cherry-pick in a loop.

-- 
Thomas Rast
trast@{inf,student}.ethz.ch
Previous: Michael WittenNext: Johannes Sixt
Message 3 of 9 in “Git Bug - diff in commit message.”
  1. anikeySep 8, 2011
  2. Michael WittenSep 8, 2011
  3. Thomas RastSep 9, 2011
  4. Johannes SixtSep 9, 2011
  5. Junio C HamanoSep 9, 2011
  6. Junio C HamanoOct 11, 2011
  7. Martin von ZweigbergkOct 24, 2011
  8. Junio C HamanoOct 25, 2011
  9. Junio C HamanoSep 8, 2011

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.