On Wednesday 03 August 2011 17:32:09 Jon Seymour wrote: > On Thu, Aug 4, 2011 at 1:03 AM, Jon Seymour <jon.seymour@xxxxxxxxx> wrote: > rm -f "$GIT_DIR/head-name" && > > > - > > - rm -f "$GIT_DIR/BISECT_START" > > + rm -f "$GIT_DIR/BISECT_START" && > > + git update-ref -d BISECT_HEAD > > } > > I'll reverse the order of the last two statements and restore the blank > line... Great! It would be nice too if the "git update-ref" call that creates the ref in the first place was after the creation of the $GIT_DIR/BISECT_START file. This way we start by creating the $GIT_DIR/BISECT_START file and we finish by deleting it. That is change this: ... + test "$mode" != "--no-checkout" || + git update-ref --no-deref BISECT_HEAD "$start_head" && echo "$start_head" >"$GIT_DIR/BISECT_START" && into something like this: echo "$start_head" >"$GIT_DIR/BISECT_START" && + { + test "$mode" != "--no-checkout" || + git update-ref --no-deref BISECT_HEAD "$start_head" + } && ... Thanks, Christian. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html