On Sun, Jul 31, 2011 at 1:55 PM, Jon Seymour <jon.seymour@xxxxxxxxx> wrote:
>
> @@ -69,13 +71,18 @@ bisect_start() {
> orig_args=$(git rev-parse --sq-quote "$@")
> bad_seen=0
> eval=''
> + BISECT_UPDATE_REF=
> while [ $# -gt 0 ]; do
> arg="$1"
> case "$arg" in
> --)
> - shift
> - break
> - ;;
> + shift; break ;;
Please don't change this. We try to avoid having many instructions on
the same line like this.
> + --no-checkout)
> + BISECT_UPDATE_REF=HEAD; shift ;;
Perhaps:
test -z "$BISECT_UPDATE_REF" && BISECT_UPDATE_REF=HEAD
so that "--update-ref=someref --no-checkout" works the same as
"--no-checkout --update-ref=someref".
> + --update-ref=*)
> + BISECT_UPDATE_REF=${arg#--update-ref=}; shift ;;
> + --*)
> + die "$(eval_gettext "unrecognised option: '\$arg'")" ;;
> *)
> rev=$(git rev-parse -q --verify "$arg^{commit}") || {
> test $has_double_dash -eq 1 &&
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