Re: [PATCH v2 2/7] bisect: replace hardcoded "bad|good" by variables
- From
Matthieu Moy <matthieu.moy@grenoble-inp.fr>
- Date
- Jun 22, 2015, 13:47 UTC
- Message-ID
- <vpqvbefc0sp.fsf@anie.imag.fr>
- In-Reply-To
- <134997869.703022.1434976943783.JavaMail.zimbra@ensimag.grenoble-inp.fr>
Antoine Delaite <antoine.delaite@ensimag.grenoble-inp.fr> writes:
Show 9 quoted lines
> Matthieu Moy <Matthieu.Moy@grenoble-inp.fr> wrote: > >>Being an acceptable ref name is a constraint you have to check (Junio >>already mentionned check-ref-format). I think quoting variables makes >>sense too > > I don't get how 'git check-ref-format' works exactly. It says it needs > at least one slash in the ref name. So it would not be good for bisect > terms.
At some point, refs/bisect/$good and refs/bisect/$bad-$sha1 will become refs, so you should check refs/bisect/$good and refs/bisect/$bad with check-ref-format (I think the -$sha1 suffix will be accepted by construction).
-- Matthieu Moy http://www-verimag.imag.fr/~moy/