Re: "git bisect run make" adequate to locate first unbuildable commit?
On Fri, 9 Feb 2018, Philip Oakley wrote: > From: "Robert P. J. Day" > > On Fri, 9 Feb 2018, Philip Oakley, CEng MIET wrote: > (apologies for using the fancy letters after the name ID...) > > > >> From: "Robert P. J. Day" > >> > > >> > writing a short tutorial on "git bisect" and, all the details > >> > of special exit code 125 aside, if one wanted to locate the > >> > first unbuildable commit, would it be sufficient to just run? > >> > > >> > $ git bisect run make > >> > > >> > as i read it, make returns either 0, 1 or 2 so there doesn't > >> > appear to be any possibility of weirdness with clashing with a > >> > 125 exit code. am i overlooking some subtle detail here i > >> > should be aware of? thanks. > >> > > >> > rday > >> > >> In the spirit of pedanticism, one should also clarify the word > >> "first", in that it's not a linear search for _an_ unbuildable > >> commit, but that one is looking for the transition between an > >> unbroken sequence of unbuildable commits, which transitions to > >> buildable commits, and its the transition that is sought. (there > >> could be many random unbuildable commits within a sequence in > >> some folks' processes!) > > > > quite so, i should have been more precise. > > > > rday > > The other two things that may be happening (in the wider bisect > discussion) that I've heard of are: > 1. there may be feature branches that bypass the known good starting >commit, which can cause understanding issues as those side >branches that predate the start point are also considered >potential bu commits. ... snip ... sorry, i should have replied to this post since my last post directly addresses this scenario -- i believe this is precisely what you were suggesting: ... 5 commits ... (feature branch) / ^ /\ v \ A <-- B <-- <-- D <-- E <-- F <-- i'm still curious ... if one doesn't get into skipping, what is the "git rev-list" command to identify all of the possible revisions to be tested here? isn't it just: $ git rev-list ^ or was there somthing more subtle i was missing? sorry if someone already explained this, i'll go back later and read the replies in more excruciating detail. rday p.s. i'll take a closer look at the "--bisect*" options to "git rev-list" shortly.
Re: "git bisect run make" adequate to locate first unbuildable commit?
> > 1. there may be feature branches that bypass the known good starting > >commit, which can cause understanding issues as those side > >branches that predate the start point are also considered > >potential bu commits. > > ok, but let's make sure i understand what defines a possible commit > that "introduces" the bug. if i define two bisection commits > and , then i've always assumed that what i'm after is a commit > for which: > > 1) is reachable from > 2) is reachable from > > this seems fairly obvious. Well, maybe _you_ are after such a commit, but bisect is after a commit for which 1) is reachable from (i.e. the same as your first point) 2) is not reachable from (which is not the same as your second point, notably when it comes to commits on side branches that branched off before and got merged later). > now, as you suggest, it's possible that the > "bug" was introduced on a feature branch that bypasses my choice of > but, at *some* point, that feature branch would have to be > merged to the point where it was now reachable from and, in the > context of bisection, *that* merge commit would represent where the > bug was introduced, no? No. Consider this piece of history: v v --a---b---C---d---e---M---k---L-- \ / f---g---H---i---j ^ first bad Then the command 'git bisect start L C' will first consider the following as "possible commit that introduces the bug": d e f g H i j M k L (IOW all commits listed by 'git log ^C L') and will then systematically narrow down until it will find commit H as the transition from good to bad.
Re: "git bisect run make" adequate to locate first unbuildable commit?
On Fri, 9 Feb 2018, Philip Oakley wrote: > From: "Robert P. J. Day" > > On Fri, 9 Feb 2018, Philip Oakley, CEng MIET wrote: > (apologies for using the fancy letters after the name ID...) > > > >> From: "Robert P. J. Day" > >> > > >> > writing a short tutorial on "git bisect" and, all the details > >> > of special exit code 125 aside, if one wanted to locate the > >> > first unbuildable commit, would it be sufficient to just run? > >> > > >> > $ git bisect run make > >> > > >> > as i read it, make returns either 0, 1 or 2 so there doesn't > >> > appear to be any possibility of weirdness with clashing with a > >> > 125 exit code. am i overlooking some subtle detail here i > >> > should be aware of? thanks. > >> > >> In the spirit of pedanticism, one should also clarify the word > >> "first", in that it's not a linear search for _an_ unbuildable > >> commit, but that one is looking for the transition between an > >> unbroken sequence of unbuildable commits, which transitions to > >> buildable commits, and its the transition that is sought. (there > >> could be many random unbuildable commits within a sequence in > >> some folks' processes!) > > > > quite so, i should have been more precise. > > > > rday > > The other two things that may be happening (in the wider bisect > discussion) that I've heard of are: > 1. there may be feature branches that bypass the known good starting >commit, which can cause understanding issues as those side >branches that predate the start point are also considered >potential bu commits. ok, but let's make sure i understand what defines a possible commit that "introduces" the bug. if i define two bisection commits and , then i've always assumed that what i'm after is a commit for which: 1) is reachable from 2) is reachable from this seems fairly obvious. now, as you suggest, it's possible that the "bug" was introduced on a feature branch that bypasses my choice of but, at *some* point, that feature branch would have to be merged to the point where it was now reachable from and, in the context of bisection, *that* merge commit would represent where the bug was introduced, no? rday
Re: "git bisect run make" adequate to locate first unbuildable commit?
From: "Robert P. J. Day" On Fri, 9 Feb 2018, Philip Oakley, CEng MIET wrote: (apologies for using the fancy letters after the name ID...) From: "Robert P. J. Day" > > writing a short tutorial on "git bisect" and, all the details of > special exit code 125 aside, if one wanted to locate the first > unbuildable commit, would it be sufficient to just run? > > $ git bisect run make > > as i read it, make returns either 0, 1 or 2 so there doesn't appear > to be any possibility of weirdness with clashing with a 125 exit code. > am i overlooking some subtle detail here i should be aware of? thanks. > > rday In the spirit of pedanticism, one should also clarify the word "first", in that it's not a linear search for _an_ unbuildable commit, but that one is looking for the transition between an unbroken sequence of unbuildable commits, which transitions to buildable commits, and its the transition that is sought. (there could be many random unbuildable commits within a sequence in some folks' processes!) quite so, i should have been more precise. rday The other two things that may be happening (in the wider bisect discussion) that I've heard of are: 1. there may be feature branches that bypass the known good starting commit, which can cause understanding issues as those side branches that predate the start point are also considered potential bu commits. 2. if you just want the first parent check for the bad commit point, that mark the second parents of merges as being good. Also, I'd expect that the skipped commits aren't 'counted' (too hard?) for the bisect algorithm's reporting. https://stackoverflow.com/questions/5638211/how-do-you-get-git-bisect-to-ignore-merged-branches contains a number of the ideas.. Philip
Re: "git bisect run make" adequate to locate first unbuildable commit?
From: "Robert P. J. Day" writing a short tutorial on "git bisect" and, all the details of special exit code 125 aside, if one wanted to locate the first unbuildable commit, would it be sufficient to just run? $ git bisect run make as i read it, make returns either 0, 1 or 2 so there doesn't appear to be any possibility of weirdness with clashing with a 125 exit code. am i overlooking some subtle detail here i should be aware of? thanks. rday In the spirit of pedanticism, one should also clarify the word "first", in that it's not a linear search for _an_ unbuildable commit, but that one is looking for the transition between an unbroken sequence of unbuildable commits, which transitions to buildable commits, and its the transition that is sought. (there could be many random unbuildable commits within a sequence in some folks' processes!) -- Philip
Re: "git bisect run make" adequate to locate first unbuildable commit?
On Fri, 9 Feb 2018, Philip Oakley, CEng MIET wrote: > From: "Robert P. J. Day" > > > > writing a short tutorial on "git bisect" and, all the details of > > special exit code 125 aside, if one wanted to locate the first > > unbuildable commit, would it be sufficient to just run? > > > > $ git bisect run make > > > > as i read it, make returns either 0, 1 or 2 so there doesn't appear > > to be any possibility of weirdness with clashing with a 125 exit code. > > am i overlooking some subtle detail here i should be aware of? thanks. > > > > rday > > In the spirit of pedanticism, one should also clarify the word > "first", in that it's not a linear search for _an_ unbuildable > commit, but that one is looking for the transition between an > unbroken sequence of unbuildable commits, which transitions to > buildable commits, and its the transition that is sought. (there > could be many random unbuildable commits within a sequence in some > folks' processes!) quite so, i should have been more precise. rday
Re: "git bisect run make" adequate to locate first unbuildable commit?
On Fri, Feb 9, 2018 at 2:20 PM, Robert P. J. Day wrote: > > writing a short tutorial on "git bisect" and, all the details of > special exit code 125 aside, if one wanted to locate the first > unbuildable commit, would it be sufficient to just run? > > $ git bisect run make > > as i read it, make returns either 0, 1 or 2 so there doesn't appear > to be any possibility of weirdness with clashing with a 125 exit code. > am i overlooking some subtle detail here i should be aware of? thanks. I think you are not overlooking anything.
"git bisect run make" adequate to locate first unbuildable commit?
writing a short tutorial on "git bisect" and, all the details of special exit code 125 aside, if one wanted to locate the first unbuildable commit, would it be sufficient to just run? $ git bisect run make as i read it, make returns either 0, 1 or 2 so there doesn't appear to be any possibility of weirdness with clashing with a 125 exit code. am i overlooking some subtle detail here i should be aware of? thanks. rday