<feed xmlns='http://www.w3.org/2005/Atom'>
<title>git/git-bisect.sh, branch v2.3.6</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/git/git.git/
</subtitle>
<id>https://git.shady.money/git/atom?h=v2.3.6</id>
<link rel='self' href='https://git.shady.money/git/atom?h=v2.3.6'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/'/>
<updated>2014-12-29T19:19:55Z</updated>
<entry>
<title>bisect: parse revs before passing them to check_expected_revs()</title>
<updated>2014-12-29T19:19:55Z</updated>
<author>
<name>Christian Couder</name>
<email>chriscool@tuxfamily.org</email>
</author>
<published>2014-12-25T18:25:32Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=6bc02d5627358f6a264c5dbb159b5a9aeb44ebf4'/>
<id>urn:sha1:6bc02d5627358f6a264c5dbb159b5a9aeb44ebf4</id>
<content type='text'>
When running for example "git bisect bad HEAD" or
"git bisect good master", the parameter passed to
"git bisect (bad|good)" has to be parsed into a
commit hash before checking if it is the expected
commit or not.

We could do that in is_expected_rev() or in
check_expected_revs(), but it is already done in
bisect_state(). Let's just store the hash values
that result from this parsing, and then reuse
them after all the parsing is done.

This way we can also use a for loop over these
values to call bisect_write() on them, instead of
using eval.

Signed-off-by: Christian Couder &lt;chriscool@tuxfamily.org&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>grammofix in user-facing messages</title>
<updated>2014-09-02T19:00:30Z</updated>
<author>
<name>Alex Henrie</name>
<email>alexhenrie24@gmail.com</email>
</author>
<published>2014-08-30T19:56:01Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=ad5fe3771b50f3862130d150b5cc8bb316ef6f1c'/>
<id>urn:sha1:ad5fe3771b50f3862130d150b5cc8bb316ef6f1c</id>
<content type='text'>
Signed-off-by: Alex Henrie &lt;alexhenrie24@gmail.com&gt;
Acked-by: Johan Herland &lt;johan@herland.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>git-bisect.sh: avoid "test &lt;cond&gt; -a/-o &lt;cond&gt;"</title>
<updated>2014-06-09T21:47:07Z</updated>
<author>
<name>Elia Pinto</name>
<email>gitter.spiros@gmail.com</email>
</author>
<published>2014-06-06T14:55:50Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=c82af12a1bde56fd7c3b984688ef7fe9f4c44b6e'/>
<id>urn:sha1:c82af12a1bde56fd7c3b984688ef7fe9f4c44b6e</id>
<content type='text'>
The construct is error-prone; "test" being built-in in most modern
shells, the reason to avoid "test &lt;cond&gt; &amp;&amp; test &lt;cond&gt;" spawning
one extra process by using a single "test &lt;cond&gt; -a &lt;cond&gt;" no
longer exists.

Signed-off-by: Elia Pinto &lt;gitter.spiros@gmail.com&gt;
Reviewed-by: Matthieu Moy &lt;Matthieu.Moy@imag.fr&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>git-bisect.sh: fix a few style issues</title>
<updated>2014-03-04T02:29:34Z</updated>
<author>
<name>Jacopo Notarstefano</name>
<email>jacopo.notarstefano@gmail.com</email>
</author>
<published>2014-03-04T00:21:43Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=305a233c98b2cc2f779789b7003544bbbb93fc14'/>
<id>urn:sha1:305a233c98b2cc2f779789b7003544bbbb93fc14</id>
<content type='text'>
Redirection operators should have a space before them, but not after them.

Signed-off-by: Jacopo Notarstefano &lt;jacopo.notarstefano@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>typofixes: fix misspelt comments</title>
<updated>2013-11-12T17:24:27Z</updated>
<author>
<name>Masanari Iida</name>
<email>standby24x7@gmail.com</email>
</author>
<published>2013-11-12T15:17:42Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=382d20e3ebc69ad20f54e71fc5f6ca17baa23973'/>
<id>urn:sha1:382d20e3ebc69ad20f54e71fc5f6ca17baa23973</id>
<content type='text'>
Signed-off-by: Masanari Iida &lt;standby24x7@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>bisect: Fix log output for multi-parent skip ranges</title>
<updated>2013-05-22T22:40:43Z</updated>
<author>
<name>Torstein Hegge</name>
<email>hegge@resisty.net</email>
</author>
<published>2013-05-22T22:27:53Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=7358a672b2c9df7473961e66e18714963b2bc107'/>
<id>urn:sha1:7358a672b2c9df7473961e66e18714963b2bc107</id>
<content type='text'>
The bisect log output of skipped commits introduced in f989cac "bisect:
Log possibly bad, skipped commits at bisection end" should obtain the range of
skipped commits from

    git rev-list bad --not good-1 good-2

not

    git rev-list bad --not good-1 --not good-2

when the skipped range contains a merge with good points in each parent.

Signed-off-by: Torstein Hegge &lt;hegge@resisty.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>bisect: Log possibly bad, skipped commits at bisection end</title>
<updated>2013-04-23T16:09:44Z</updated>
<author>
<name>Torstein Hegge</name>
<email>hegge@resisty.net</email>
</author>
<published>2013-04-22T21:02:29Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=f989cac9581ee423457c02f67d6bf31348ae6b56'/>
<id>urn:sha1:f989cac9581ee423457c02f67d6bf31348ae6b56</id>
<content type='text'>
If the bisection completes with only skipped commits left to as possible
first bad commit, output the list of possible first bad commits to human
readers of the bisection log.

Signed-off-by: Torstein Hegge &lt;hegge@resisty.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>bisect: Store first bad commit as comment in log file</title>
<updated>2013-04-15T16:05:42Z</updated>
<author>
<name>Torstein Hegge</name>
<email>hegge@resisty.net</email>
</author>
<published>2013-04-13T15:22:57Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=a7f8b8ac945b32d00086c6e3d2fe7b0d00924442'/>
<id>urn:sha1:a7f8b8ac945b32d00086c6e3d2fe7b0d00924442</id>
<content type='text'>
When bisect successfully finds a single revision, the first bad commit
should be shown to human readers of 'git bisect log'.

This resolves the apparent disconnect between the bisection result and
the log when a bug reporter says "I know that the first bad commit is
$rev, as you can see from $(git bisect log)".

Signed-off-by: Torstein Hegge &lt;hegge@resisty.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'js/bisect-no-checkout'</title>
<updated>2011-10-18T04:37:09Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2011-10-18T04:37:09Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=a9af6c451d68267a605b7b7385cdffa939b167b2'/>
<id>urn:sha1:a9af6c451d68267a605b7b7385cdffa939b167b2</id>
<content type='text'>
* js/bisect-no-checkout:
  bisect: fix exiting when checkout failed in bisect_start()
</content>
</entry>
<entry>
<title>bisect: fix exiting when checkout failed in bisect_start()</title>
<updated>2011-09-21T11:55:15Z</updated>
<author>
<name>Christian Couder</name>
<email>chriscool@tuxfamily.org</email>
</author>
<published>2011-09-21T05:17:24Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=1acf11717f01b542e099544796f00df567a66e90'/>
<id>urn:sha1:1acf11717f01b542e099544796f00df567a66e90</id>
<content type='text'>
Commit 4796e823 ("bisect: introduce --no-checkout support into porcelain." Aug 4 2011)
made checking out the branch where we started depends on the "checkout" mode. But
unfortunately it lost the "|| exit" part after the checkout command.

As it makes no sense to continue if the checkout failed and as people have already
complained that the error message given when we just exit in this case is not clear, see:

http://thread.gmane.org/gmane.comp.version-control.git/180733/

this patch adds a "|| die &lt;hopefully clear message&gt;" part after the checkout command.

Signed-off-by: Christian Couder &lt;chriscool@tuxfamily.org&gt;
Acked-by: Jon Seymour &lt;jon.seymour@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
</feed>
