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

Re: [PATCH] t5705: Use the correct file:// URL

From
Junio C Hamano <gitster@pobox.com>
Date
Nov 13, 2014, 16:37 UTC
Message-ID
<xmqqtx232h0j.fsf@gitster.dls.corp.google.com>
In-Reply-To
<20141113091724.GB8329@peff.net>
Jeff King <peff@peff.net> writes:
Show 13 quoted lines
> On Thu, Nov 13, 2014 at 08:36:07AM +0100, Torsten Bögershausen wrote:
>
>> A URL like file;//. is (no longer) supported by Git:
>> Typically there is no host, and RFC1738 says that file:///<path>
>> should be used.
>> 
>> Update t5705 to use a working URL.
>
> Interesting. This looks like it was unintentionally lost in c59ab2e
> (connect.c: refactor url parsing, 2013-11-28). Given RFC1738, and that
> this is the first notice of it (and that it is not even a real use case,
> but something questionable in the test script), it's probably OK to
> declare the syntax dead and not treat it like a regression.

Yeah, I tend to agree. I do not think there is any <scheme>:// that lets you say things relative to something that is not "root" of the namespace, and it was a bug the old test depended on that we did not notice it as an error when "file://." tried to refer to "the current directory".

Previous: Jeff King
Message 3 of 3 in “t5705: Use the correct file:// URL”
  1. t5705: Use the correct file:// URLTorsten Bögershausen, Nov 13, 2014
  2. Jeff KingNov 13, 2014
  3. Junio C HamanoNov 13, 2014

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.