git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCHv2 0/5] Travis CI: don't run the test suite as root in the 32 bit Linux build

From
Jeff King <peff@peff.net>
Date
Jan 29, 2018, 18:23 UTC
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:
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
Previous: SZEDER Gábor
Message 7 of 7 in “[PATCHv2 0/5] Travis CI: don't run the test suite as root in the 32 bit Linux build”
  1. SZEDER GáborJan 29, 2018
  2. 1/5 travis-ci: use 'set -x' for the commands under 'su' in the 32 bit Linux buildSZEDER Gábor, Jan 29, 2018
  3. 3/5 travis-ci: don't repeat the path of the cache directorySZEDER Gábor, Jan 29, 2018
  4. 5/5 travis-ci: don't fail if user already exists on 32 bit Linux build jobSZEDER Gábor, Jan 29, 2018
  5. 2/5 travis-ci: use 'set -e' in the 32 bit Linux build jobSZEDER Gábor, Jan 29, 2018
  6. 4/5 travis-ci: don't run the test suite as root in the 32 bit Linux buildSZEDER Gábor, Jan 29, 2018
  7. Jeff KingJan 29, 2018

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.