Re: [PATCHv2 0/5] Travis CI: don't run the test suite as root in the 32 bit Linux build
On Mon, Jan 29, 2018 at 06:17:08PM +0100, SZEDER Gábor wrote:
Show 12 quoted lines
> 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