> On 22 May 2016, at 17:35, Christian Couder <christian.couder@xxxxxxxxx> wrote:
>
> On Sun, May 22, 2016 at 1:00 PM, <larsxschneider@xxxxxxxxx> wrote:
>
> [...]
>
>> +#
>> +# Run Git bisect
>> +#
>> +run_bisect () {
>> + TEST_SCRIPT=$1
>> + BAD_REV=$2
>> + GOOD_RV=$3
>> + TMPDIR=$(mktemp -d -t "ci-report-bisect-XXXXXX" 2>/dev/null)
>> + cat > "$TMPDIR/bisect-run.sh" <<EOF
>> +
>> +EOF
>> + chmod +x "$TMPDIR/bisect-run.sh"
>> + git bisect start $BAD_REV $GOOD_RV
>> + git bisect run "$TMPDIR/bisect-run.sh"
>> + if test -e ./t/$TEST_SCRIPT.sh && make --jobs=2 >/dev/null 2>&1
>> + then
>> + cd t && ./$TEST_SCRIPT.sh >/dev/null 2>&1
>> + else
>> + # If the test file does not exist or the build fails then tell
>> + # Git bisect to skip the commit.
>> + exit 125
>> + fi
>
> Shouldn't all the above "if ... fi" be in the here document creating
> "$TMPDIR/bisect-run.sh"?
Ohh. Absolutely... I wonder what happened. I'll post a v2.
Thank you,
Lars
>
>> + git bisect reset >/dev/null 2>&1
>> +}
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html