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

Re: Marking commits as transitory for git bisect?

From
Junio C Hamano <gitster@pobox.com>
Date
Dec 2, 2009, 16:49 UTC
Message-ID
<7vhbs9jp7n.fsf@alter.siamese.dyndns.org>
In-Reply-To
<87tyw9wsfr.fsf@lola.goethe.zz>
David Kastrup <dak@gnu.org> writes:
> Further comments/insights?

Just make sure that you state in the log message of the commits in the series something like:

 - The way the series is split for easier review has to break X and Y
   and this commit does not even compile;
 - This is the second half of restructuring, and it compiles again;
 - Now A is fully restructured, test X works again, but Y is still broken;
 - This finally completes the series and everything works again.

for _human consumption_. Bisect is not the only thing that needs to know about these commits that are intentionally broken. The reviewers and people who test before accepting the history with these commits in their history need to know about the known breakages, too.

Because you are talking about breakages that are _known_ when you make the series, you can afford to follow this recommendation. Object replacement mechanism is _not_ the answer---it's whole point is that you can add it as an afterthought by (virtually) squashing the series into one.

This way, the initial reviewers and a person who happened to hit such an intentionally broken commit during bisect can deal with the situation exactly the same way. I.e. with "git log".

Previous: David Kastrup
Message 4 of 4 in “Marking commits as transitory for git bisect?”
  1. David KastrupDec 2, 2009
  2. Michael J GruberDec 2, 2009
  3. David KastrupDec 2, 2009
  4. Junio C HamanoDec 2, 2009

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.