From: Jeff King <peff@peff.net>
Date: Mon, 29 Jan 2018 18:23:16 GMT
Subject: Re: [PATCHv2 0/5] Travis CI: don't run the test suite as root in the 32 bit Linux build
Message-ID: <20180129182316.GD4185@sigill.intra.peff.net>
In-Reply-To: <20180129171713.17471-1-szeder.dev@gmail.com>

On Mon, Jan 29, 2018 at 06:17:08PM +0100, SZEDER Gábor wrote:

> This version addresses Peff comments:
> 
>   - Updated the commit message to discuss the broken && chain and its
>     consequences in patch 2.
>   - Fixed quoting of the 'su' command snippet and updated the commit
>     message in patch 3.
>   - Squelch error from 'test "$(id -u $CI_USER)"' and add debug
>     message in patch 5.
> 
> Additionally, it's rebased onto current master (I branched off v1 from
> 'sg/travis-build-during-script-phase' to avoid a conflict, but Junio
> applied it on top of current master).

Thanks, this all looks good to me.

-Peff