Re: [PATCH v9 4/5] bisect: add the terms old/new

From: Matthieu Moy <Matthieu.Moy@grenoble-inp.fr>
To: Christian Couder <christian.couder@gmail.com>
Cc: Junio C Hamano <gitster@pobox.com>, git <git@vger.kernel.org>,
	Antoine Delaite <antoine.delaite@ensimag.grenoble-inp.fr>,
	louis--alexandre stuber 
	<louis--alexandre.stuber@ensimag.grenoble-inp.fr>,
	Christian Couder <chriscool@tuxfamily.org>,
	Thomas Nguy <thomasxnguy@gmail.com>,
	Valentin Duperray <valentinduperray@gmail.com>,
	Louis Stuber <stuberl@ensimag.grenoble-inp.fr>
Subject: Re: [PATCH v9 4/5] bisect: add the terms old/new
Date: Fri, 26 Jun 2015 09:00:55 +0200	[thread overview]
Message-ID: <vpqoak3x8bc.fsf@anie.imag.fr> (raw)
In-Reply-To: <CAP8UFD1EWYzg7-O4BDgUTQPJOoo6TEAF49amakgfR8K6CX886A@mail.gmail.com> (Christian Couder's message of "Fri, 26 Jun 2015 06:11:57 +0200")

Christian Couder <christian.couder@gmail.com> writes:

> So perhaps you could use a function like:
>
> write_bisect_terms() {
>       if test ! -s "$GIT_DIR/BISECT_TERMS"
>       then
>             echo "$NAME_BAD" >"$GIT_DIR/BISECT_TERMS" &&
>             echo "$NAME_GOOD" >>"$GIT_DIR/BISECT_TERMS"
>       fi
> }

I already had it in the last patch, but reworked the code to introduce
it earlier, and never >"$GIT_DIR/BISECT_TERMS" outside this function.

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/