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

Re: Opinions on bug fix organisation

From
Martin Langhoff <martin.langhoff@gmail.com>
Date
May 16, 2007, 21:51 UTC
Message-ID
<46a038f90705161451v3bf2c04agb95c03f14f236e76@mail.gmail.com>
In-Reply-To
<200705161138.30134.andyparkins@gmail.com>
On 5/16/07, Andy Parkins <andyparkins@gmail.com> wrote:
> 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.

I prefer just letting history show what happened, rather than try to get too smart about it ;-) -- and use branches and merges for experimental or feature work. Once a feature or experimental branch is merged into master, further work happens on master (unless there are other reasons for it to be maintained).

Bugfixes are part of the life of the maint and master branches. Imagine your "option 2" being used to maintain git's maint branch. Some bugs live in the code for 6 months. The merge graph would be unreadable... and generally the project history would be really hard to make sense of.

Most "special" practices around branches kind-of work in the minimalistic case, but break down badly in real-life sized projects...

cheers,
martin
Previous: Johannes SixtNext: Jan Hudec
Message 7 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.