Re: [PATCH 0/5] bisect: Add support for a --no-checkout option.
- From
Jon Seymour <jon.seymour@gmail.com>
- Date
- Aug 2, 2011, 21:27 UTC
- Message-ID
- <CAH3Anrryu5k_3ykC=PzsPoik_0+HaG+5d9HdW2qsJTmS41j+oQ@mail.gmail.com>
- In-Reply-To
- <7vvcufem32.fsf@alter.siamese.dyndns.org>
On Wed, Aug 3, 2011 at 3:53 AM, Junio C Hamano <gitster@pobox.com> wrote:
Show 14 quoted lines
> Jon Seymour <jon.seymour@gmail.com> writes: > >>> Then, the next series can add a --bisect-head option (instead of >>> --update-ref) to allow the head to be varied (defaulting to >>> BISECT_HEAD) if not specified. >> >> To clarify: --bisect-head would default to HEAD if mode is checkout, >> to BISECT_HEAD if the mode is update-ref. > > I am not sure if we ever want to allow updating anything but HEAD under > checkout mode, so I would rather think --bisect-head is _always_ invalid > under that mode. IOW, as I said in the previous message, these two are not > independent options. >
I agree. In fact, I like your idea of a BISECT_HEAD for --no-checkout, and I don't see any need (even in the future) to allow this to be configurable [ except perhaps if someone thinks doing a --no-checkout with HEAD is actually useful ].
jon.