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

Re: [PATCH 0/5] bisect: Add support for a --no-checkout option.

From
Christian Couder <chriscool@tuxfamily.org>
Date
Jul 30, 2011, 13:48 UTC
Message-ID
<201107301548.09815.chriscool@tuxfamily.org>
In-Reply-To
<1312014511-7157-1-git-send-email-jon.seymour@gmail.com>
Hi,
On Saturday 30 July 2011 10:28:26 Jon Seymour wrote:
> For some bisection tasks, checking out the commit at each stage of the
> bisection process is unecessary or undesirable.
> 
> This series adds support for a --no-checkout[=<ref>] option to git-bisect.

Sorry but I didn't reply to your previous email when you asked about a "--no- checkout[=<ref>]" compromise. I thought that Junio would reply and then I forgot about it.

My opinion is that if you really want to be able to use another ref, then there should be a special "--update-ref=<ref>" or "--use-ref=<ref>" option that is different from "--no-checkout".

"--no-checkout" looks like a boolean argument. And "--no-checkout[=<ref>]" may make the user think that this option will not checkout <ref>, and then it leads to the confusing question "but why would it checkout this f&#@ing ref in the first place?".

Thanks, Christian.

Previous: Jon SeymourNext: Jon Seymour
Message 14 of 26 in “bisect: Add support for a --no-checkout option.”
  1. 0/5 bisect: Add support for a --no-checkout option.Jon Seymour, Jul 30, 2011
  2. 1/5 bisect: add tests to document expected behaviour in presence of broken trees.Jon Seymour, Jul 30, 2011
  3. 2/5 bisect: introduce support for --no-checkout=<ref> option.Jon Seymour, Jul 30, 2011
  4. Christian CouderJul 30, 2011
  5. 3/5 bisect: introduce --no-checkout[=<ref>] support into porcelain.Jon Seymour, Jul 30, 2011
  6. Christian CouderJul 30, 2011
  7. Jon SeymourJul 30, 2011
  8. Christian CouderJul 31, 2011
  9. Junio C HamanoAug 1, 2011
  10. Christian CouderAug 1, 2011
  11. 4/5 bisect: add tests for the --no-checkout option.Jon Seymour, Jul 30, 2011
  12. 5/5 bisect: add documentation for --no-checkout[=<ref>] option.Jon Seymour, Jul 30, 2011
  13. Jon SeymourJul 30, 2011
  14. Christian CouderJul 30, 2011
  15. Jon SeymourJul 30, 2011
  16. Christian CouderJul 30, 2011
  17. Junio C HamanoAug 1, 2011
  18. Christian CouderAug 1, 2011
  19. Jon SeymourAug 1, 2011
  20. Junio C HamanoAug 1, 2011
  21. Jon SeymourAug 1, 2011
  22. Junio C HamanoAug 1, 2011
  23. Jon SeymourAug 2, 2011
  24. Jon SeymourAug 2, 2011
  25. Junio C HamanoAug 2, 2011
  26. Jon SeymourAug 2, 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.