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

Re: [PATCH] t5813: avoid creating urls that break on cygwin

From
Jeff King <peff@peff.net>
Date
Nov 9, 2015, 15:45 UTC
Message-ID
<20151109154501.GA27224@sigill.intra.peff.net>
In-Reply-To
<20151108095400.GA9448@spirit>
On Sun, Nov 08, 2015 at 10:54:04AM +0100, Dennis Kaarsemaker wrote:
> The fake ssh used by this test simply strips ssh://host from the url,
> leaving paths behind that start with //, which cygwin interprets as UNC
> paths, causing the test to fail.

I found the first sentence a little misleading. It is git itself that strips the URL, isn't it? The problem is that we are feeding a URL with a bogus path, which the fake ssh then tries to access (but in a way that happens to work on Unix systems).

-Peff
Previous: Dennis KaarsemakerNext: Dennis Kaarsemaker
Message 13 of 16 in “Bug: t5813 failing on Cygwin”
  1. Adam DinwoodieNov 7, 2015
  2. Jeff KingNov 7, 2015
  3. Adam DinwoodieNov 7, 2015
  4. Dennis KaarsemakerNov 7, 2015
  5. Ramsay JonesNov 7, 2015
  6. Ramsay JonesNov 7, 2015
  7. Dennis KaarsemakerNov 7, 2015
  8. Ramsay JonesNov 8, 2015
  9. Torsten BögershausenNov 8, 2015
  10. Adam DinwoodieNov 7, 2015
  11. Jeff KingNov 8, 2015
  12. t5813: avoid creating urls that break on cygwinDennis Kaarsemaker, Nov 8, 2015
  13. Jeff KingNov 9, 2015
  14. t5813: avoid creating urls that break on cygwinDennis Kaarsemaker, Nov 9, 2015
  15. Jeff KingNov 9, 2015
  16. Ramsay JonesNov 8, 2015

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.