Re: [PATCH] bisect: revise manpage

2015-06-26 Thread Junio C Hamano
Michael Haggerty writes: > By the way, when I was revising the text two things occurred to me that > have probably been discussed to death elsewhere but let me mention them > anyway: > > 1. I found it confusing that `git bisect terms` lists its arguments in > the order ` `. I think that listing t

Re: [PATCH] bisect: revise manpage

2015-06-26 Thread Junio C Hamano
Michael Haggerty writes: >>> Eventually there will be no more revisions left to bisect, and the >>> command will print out a description of the first bad commit. The >>> reference `refs/bisect/bad` created by bisect will point at that >>> commit. > > I agree that is better. > >> For the last sent

Re: [PATCH] bisect: revise manpage

2015-06-26 Thread Matthieu Moy
Michael Haggerty writes: > I didn't like this example so much because (1) the code snippet is > pretty trivial, and (2) the explanation afterwards is more of a general > explanation of `git bisect` than a description of this particular > example. I agree that the explanations were redundant. I r

Re: [PATCH] bisect: revise manpage

2015-06-26 Thread Matthieu Moy
Christian Couder writes: > On Fri, Jun 26, 2015 at 4:58 PM, Michael Haggerty > wrote: > >> The reference `refs/bisect/bad` will be left pointing at that commit. > > Yeah ok. I took this one. -- Matthieu Moy http://www-verimag.imag.fr/~moy/ -- To unsubscribe from this list: send the line "uns

Re: [PATCH] bisect: revise manpage

2015-06-26 Thread Christian Couder
On Fri, Jun 26, 2015 at 4:58 PM, Michael Haggerty wrote: > On 06/26/2015 03:15 PM, Christian Couder wrote: >> On Fri, Jun 26, 2015 at 3:00 PM, Matthieu Moy >> wrote: >>> Christian Couder writes: >>> On Fri, Jun 26, 2015 at 1:30 PM, Michael Haggerty wrote: [...] > +

Re: [PATCH] bisect: revise manpage

2015-06-26 Thread Michael Haggerty
On 06/26/2015 03:15 PM, Christian Couder wrote: > On Fri, Jun 26, 2015 at 3:00 PM, Matthieu Moy > wrote: >> Christian Couder writes: >> >>> On Fri, Jun 26, 2015 at 1:30 PM, Michael Haggerty >>> wrote: >>> >>> [...] >>> +Eventually there will be no more revisions left to bisect, and the >>>

Re: [PATCH] bisect: revise manpage

2015-06-26 Thread Michael Haggerty
On 06/26/2015 02:50 PM, Matthieu Moy wrote: > Michael Haggerty writes: > >> * Remove the "Look for a fix instead of a regression in the code" >> example, as (1) it was in the "git bisect run" section, but it >> doesn't use that command, and (2) I think this usage is adequately >> explained

Re: [PATCH] bisect: revise manpage

2015-06-26 Thread Christian Couder
On Fri, Jun 26, 2015 at 3:00 PM, Matthieu Moy wrote: > Christian Couder writes: > >> On Fri, Jun 26, 2015 at 1:30 PM, Michael Haggerty >> wrote: >> >> [...] >> >>> +Eventually there will be no more revisions left to bisect, and the >>> +command will print out a description of the first bad comm

Re: [PATCH] bisect: revise manpage

2015-06-26 Thread Matthieu Moy
Christian Couder writes: > On Fri, Jun 26, 2015 at 1:30 PM, Michael Haggerty > wrote: > > [...] > >> +Eventually there will be no more revisions left to bisect, and the >> +command will print out a description of the first bad commit, and also >> +create a reference called `refs/bisect/bad` tha

Re: [PATCH] bisect: revise manpage

2015-06-26 Thread Matthieu Moy
Michael Haggerty writes: > * Remove the "Look for a fix instead of a regression in the code" > example, as (1) it was in the "git bisect run" section, but it > doesn't use that command, and (2) I think this usage is adequately > explained in the "Alternate terms" section. [...] > -* Look fo

Re: [PATCH] bisect: revise manpage

2015-06-26 Thread Christian Couder
On Fri, Jun 26, 2015 at 1:30 PM, Michael Haggerty wrote: [...] > +Eventually there will be no more revisions left to bisect, and the > +command will print out a description of the first bad commit, and also > +create a reference called `refs/bisect/bad` that points at that > +commit. This could

[PATCH] bisect: revise manpage

2015-06-26 Thread Michael Haggerty
Thoroughly revise the "git bisect" manpage, including: * Beef up the "Description" section. * Integrate the good/bad alternate terms into more of the text. * Merge the sections "Alternative terms: bisect new and bisect old" and "Alternative terms: use your own terms" into a single "Alternate