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

Re: What's cooking in git.git (Aug 2009, #05; Wed, 26)

From
Junio C Hamano <gitster@pobox.com>
Date
Aug 30, 2009, 20:31 UTC
Message-ID
<7vzl9h9h70.fsf@alter.siamese.dyndns.org>
In-Reply-To
<7vskfat07h.fsf@alter.siamese.dyndns.org>
Junio C Hamano <gitster@pobox.com> writes:
Show 16 quoted lines
> Daniel Barkalow <barkalow@iabervon.org> writes:
>
>>> There was a discussion that suggests that the use of colon ':' before vcs
>>> helper name needs to be corrected.  Nothing happened since.
>>
>> I believe the outcome of that discussion was:
>>
>>  - We want to keep supporting using regular location URLs that are URLs of 
>>    git repositories (e.g., http://git.savannah.gnu.org/cgit/xboard.git), 
>>    and we probably want to do it with a helper which runs when 
>>    run_command() is given "remote-<scheme>". I think installing hardlinks 
>>    in EXECPATH ended up being the best implementation here.
>
> That is different from what I recall.
> ...
> And from my point of view, this is what is blocking the series...
Just to make my position clear.

I brought up the issue of "should we leave it as a single colon that confuses scp-like syntax and forces us to have three extra hardlinks?" in the message I am following up to, not because I firmly am on that side of the argument, but because I thought your version of "outcome" did not match my recollection around that particular issue. I am more or less neutral myself. Avoiding the confusion seems obviously the simple and right thing to do, and I cannot think of obvious downsides of such a change, but there probably are some downsides you have in mind, and I can well imagine the issue may become "perfect is the enemy of good".

And I said "blocking the series", not in the sense that _I_ demand to change it from colon to something else, but in the sense that I recall the issue hasn't been settled in the discussion.

Previous: Junio C HamanoNext: Daniel Barkalow
Message 12 of 15 in “What's cooking in git.git (Aug 2009, #05; Wed, 26)”
  1. Junio C HamanoAug 26, 2009
  2. Shawn O. PearceAug 26, 2009
  3. Sverre RabbelierAug 27, 2009
  4. Shawn O. PearceAug 27, 2009
  5. Sverre RabbelierAug 27, 2009
  6. Brandon CaseyAug 27, 2009
  7. Junio C HamanoAug 27, 2009
  8. Brandon CaseyAug 27, 2009
  9. t/test-lib.sh: provide a shell implementation of the 'yes' utilityBrandon Casey, Aug 28, 2009
  10. Daniel BarkalowAug 30, 2009
  11. Junio C HamanoAug 30, 2009
  12. Junio C HamanoAug 30, 2009
  13. Daniel BarkalowAug 31, 2009
  14. Junio C HamanoAug 31, 2009
  15. Daniel BarkalowSep 1, 2009

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.