Re: [PATCH 2/6] t1406: prepare for the refs code to fail with BUG()

Johannes Schindelin writes: >> As discussed in this thread, tests that use t/helper/ executables >> that try to trickle BUG() codepath to ensure that these "should >> never happen" conditions are caught do need to deal with it. If >> dumping core is undesirable,

Re: [PATCH 2/6] t1406: prepare for the refs code to fail with BUG()

Hi, On Wed, 2 May 2018, Junio C Hamano wrote: > Duy Nguyen writes: > > > On Mon, Apr 30, 2018 at 12:17 AM, Johannes Schindelin > > wrote: > >> t1406 specifically verifies that certain code paths fail with a BUG: ... > >> message. > >> > >> In the

Re: [PATCH 2/6] t1406: prepare for the refs code to fail with BUG()

Duy Nguyen writes: > On Mon, Apr 30, 2018 at 12:17 AM, Johannes Schindelin > wrote: >> t1406 specifically verifies that certain code paths fail with a BUG: ... >> message. >> >> In the upcoming commit, we will convert that message to be generated

Re: [PATCH 2/6] t1406: prepare for the refs code to fail with BUG()

On Mon, Apr 30, 2018 at 12:17 AM, Johannes Schindelin wrote: > t1406 specifically verifies that certain code paths fail with a BUG: ... > message. > > In the upcoming commit, we will convert that message to be generated via > BUG() instead of die("BUG: ..."), which

Re: [PATCH 2/6] t1406: prepare for the refs code to fail with BUG()

On Tue, May 1, 2018 at 1:04 PM, Johannes Schindelin wrote: >> If SIGABRT occurs as a result of BUG(), and we know that this happens for >> certain cases, it means we have an unfixed bug. > > Not in this case: The code in question is in >

Re: [PATCH 2/6] t1406: prepare for the refs code to fail with BUG()

Hi Hannes, On Mon, 30 Apr 2018, Johannes Sixt wrote: > Am 30.04.2018 um 00:17 schrieb Johannes Schindelin: > > t1406 specifically verifies that certain code paths fail with a BUG: ... > > message. > > > > In the upcoming commit, we will convert that message to be generated via > > BUG() instead

Re: [PATCH 2/6] t1406: prepare for the refs code to fail with BUG()

Am 30.04.2018 um 00:17 schrieb Johannes Schindelin: t1406 specifically verifies that certain code paths fail with a BUG: ... message. In the upcoming commit, we will convert that message to be generated via BUG() instead of die("BUG: ..."), which implies SIGABRT instead of a regular exit code.

[PATCH 2/6] t1406: prepare for the refs code to fail with BUG()

t1406 specifically verifies that certain code paths fail with a BUG: ... message. In the upcoming commit, we will convert that message to be generated via BUG() instead of die("BUG: ..."), which implies SIGABRT instead of a regular exit code. Signed-off-by: Johannes Schindelin