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

Re: [PATCH] mingw: hot-fix t5615

From
Jeff King <peff@peff.net>
Date
Nov 11, 2016, 18:12 UTC
Message-ID
<20161111181256.gl5xu6jxqrd5cbg7@sigill.intra.peff.net>
In-Reply-To
<f4336a7fa3df8c0b426c3d01ada8d7d8f38e588a.1478881741.git.johannes.schindelin@gmx.de>
On Fri, Nov 11, 2016 at 05:29:33PM +0100, Johannes Schindelin wrote:
> That test made the incorrect assumption that the path separator character
> is always a colon. On Windows, it is a semicolon instead.

Oof, sorry about that. I remember being careful about the ";" while doing the original alt-odb work, but forgot about it while making the recent fix in 37a95862c.

The patch itself looks like obviously correct (or at least obviously in the right direction, since it sounds like more is needed).

-Peff
Previous: Junio C Hamano
Message 17 of 17 in “mingw: hot-fix t5615”
  1. mingw: hot-fix t5615Johannes Schindelin, Nov 11, 2016
  2. Junio C HamanoNov 11, 2016
  3. Johannes SixtNov 11, 2016
  4. t0021, t5615: use $PWD instead of $(pwd) in PATH-like shell variablesJohannes Sixt, Nov 11, 2016
  5. Jeff KingNov 11, 2016
  6. Junio C HamanoNov 11, 2016
  7. Johannes SchindelinNov 12, 2016
  8. Junio C HamanoNov 13, 2016
  9. Lars SchneiderNov 14, 2016
  10. Torsten BögershausenNov 14, 2016
  11. Jeff KingNov 14, 2016
  12. Ramsay JonesNov 14, 2016
  13. Junio C HamanoNov 14, 2016
  14. Johannes SchindelinNov 15, 2016
  15. Johannes SchindelinNov 16, 2016
  16. Junio C HamanoNov 16, 2016
  17. Jeff KingNov 11, 2016

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.