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

Re: Opinions on bug fix organisation

From
JHJan Hudec <bulb@ucw.cz>
Date
May 18, 2007, 19:50 UTC
Message-ID
<20070518195000.GC3327@efreet.light.src>
In-Reply-To
<200705161138.30134.andyparkins@gmail.com>
On Wed, May 16, 2007 at 11:38:28 +0100, Andy Parkins wrote:
Show 18 quoted lines
> Now, I want to make a commit that fixes that bug.  These are the options:
> 
>  * -- * -- B -- * -- * -- * -- F
> 
>  or
> 
>  * -- * -- B -- * -- * -- * -- M
>             \                 /
>              --------------- F
> 
> That is - just commit a fix or, commit the fix, "F", directly on "B" then 
> merge that fix back to HEAD with "M".
> 
> I quite like option 2 because it records intent - i.e. "I wish I could have 
> gone back and changed this revision, but I can't", but it makes a more 
> complicated history.
> 
> What do people think?
The big advantage of the later is, that if you have:
 * -- B -- * -- * -- M1
       \
        -- * -- * -- M2

You can merge the fix done on yet another branch into how many branches you need, so:

 * -- B -- * -- * ----- M1
       \               /
        -- * -- * ----/- M2
	 \           /  /
	  ----------F---

If you had the fix on one of the branches, you could only cherry-pick it to the other, but the history would not really reflect that.

-- 
						 Jan 'Bulb' Hudec <bulb@ucw.cz>
Previous: Martin Langhoff
Message 8 of 8 in “Opinions on bug fix organisation”
  1. Andy ParkinsMay 16, 2007
  2. Brian GernhardtMay 16, 2007
  3. Junio C HamanoMay 16, 2007
  4. Andy ParkinsMay 16, 2007
  5. Shawn O. PearceMay 16, 2007
  6. Johannes SixtMay 18, 2007
  7. Martin LanghoffMay 16, 2007
  8. Jan HudecMay 18, 2007

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.